----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57868/#review175231 -----------------------------------------------------------
There is a comma ',' missing after 'Modules' in security-admin/src/main/webapp/scripts/modules/globalize/message/en.js. By the way you can just update the diff rather than add files to the review request. - Colm O hEigeartaigh On May 17, 2017, 8:35 a.m., Haihui Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57868/ > ----------------------------------------------------------- > > (Updated May 17, 2017, 8:35 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-1411 > https://issues.apache.org/jira/browse/RANGER-1411 > > > Repository: ranger > > > Description > ------- > > 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. > > > 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.patch > > https://reviews.apache.org/media/uploaded/files/2017/05/16/5ef6eb5a-42c7-4aa7-8c54-7deace5ce158__RANGER-1411.patch > 0001-RANGER-1411-Permissions-tab-pages-display-are-not-re.patch > > https://reviews.apache.org/media/uploaded/files/2017/05/17/2ad83ef8-56bf-4922-83f6-8d254bdd673e__0001-RANGER-1411-Permissions-tab-pages-display-are-not-re.patch > > > Thanks, > > Haihui Xu > >
