-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54054/#review156824
-----------------------------------------------------------




security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js
 (line 323)
<https://reviews.apache.org/r/54054/#comment227142>

    Is removing the XAUtil.capitaliseFirstLetter(resource.name) intentional?



security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js
 (line 344)
<https://reviews.apache.org/r/54054/#comment227143>

    See above comment.


- Gautam Borad


On Nov. 24, 2016, 7:29 a.m., Mehul Parikh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54054/
> -----------------------------------------------------------
> 
> (Updated Nov. 24, 2016, 7:29 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, 
> Madhan Neethiraj, Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-1227
>     https://issues.apache.org/jira/browse/RANGER-1227
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Need to have UI Hints for search boxes that are presented on Reports page, 
> Policy listing page and Audit page. UI hints will be helpful for new users to 
> filter results as per their requirements.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/webapp/scripts/utils/XAUtils.js a1915cf 
>   
> security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js
>  5b7f584 
>   security-admin/src/main/webapp/scripts/views/reports/AuditLayout.js 023dcc8 
>   security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js 
> 0e2401e 
>   security-admin/src/main/webapp/styles/xa.css 1eb7b6e 
>   
> security-admin/src/main/webapp/templates/policies/RangerPolicyTableLayout_tmpl.html
>  bde6b37 
>   security-admin/src/main/webapp/templates/reports/AuditLayout_tmpl.html 
> 48467aa 
>   security-admin/src/main/webapp/templates/reports/UserAccessLayout_tmpl.html 
> df7acfb 
> 
> Diff: https://reviews.apache.org/r/54054/diff/
> 
> 
> Testing
> -------
> 
> Verified UI hints working properly on Chrome. 
> Added UI hints for :
>   Policy Listing page
>   Reports page
>   Access Audit tab
> 
> 
> Thanks,
> 
> Mehul Parikh
> 
>

Reply via email to