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



I agree with point (1). However, for point (2) I feel that "Module Details" is 
fine, as the page lists the module name + then user/group permissions. Could 
you resubmit a patch with just the first change?

- Colm O hEigeartaigh


On March 23, 2017, 8:32 a.m., Haihui Xu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/57868/
> -----------------------------------------------------------
> 
> (Updated March 23, 2017, 8:32 a.m.)
> 
> 
> Review request for ranger.
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> 1.Navigate to Ranger Web UI's Settings Permissions tab, the permissions table 
> header is Permissions,Groups,Users,Action. But the first column in the table 
> is modules(Resource Based Policies,Users/Groups,Audits,Reports...). So the 
> table header should be Modules,Groups,Users,Action.
> 2.Web UI's Settings Permissions tab, click action to edit permission, and the 
> page display bellow 'Edit Permission' is 'Policy Details' (0.6.3 version), 
> 'Module Details'(0.7.0 version ,1.0.0 version). The action is 'edit 
> permission',so 'Permission Details' or 'Module Permission Details' may be 
> more reasonable.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/webapp/scripts/modules/globalize/message/en.js 
> 7a4b82c 
>   
> security-admin/src/main/webapp/scripts/views/permissions/ModulePermsTableLayout.js
>  92c07c7 
>   
> security-admin/src/main/webapp/templates/permissions/ModulePermissionForm_tmpl.html
>  909d2fc 
> 
> 
> Diff: https://reviews.apache.org/r/57868/diff/1/
> 
> 
> Testing
> -------
> 
> Page Display, test ok.
> 
> 
> File Attachments
> ----------------
> 
> RANGER-1411-0.6.3.patch
>   
> https://reviews.apache.org/media/uploaded/files/2017/03/23/6b59ee99-c8bb-4458-badb-faf8b7e87f66__RANGER-1411-0.6.3.patch
> RANGER-1411-0.7.0.patch
>   
> https://reviews.apache.org/media/uploaded/files/2017/03/23/9af2e730-7030-42c9-8e61-8d3fbb0f53a9__RANGER-1411-0.7.0.patch
> 
> 
> Thanks,
> 
> Haihui Xu
> 
>

Reply via email to