----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47448/#review135498 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On May 26, 2016, 6:09 a.m., Mehul Parikh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47448/ > ----------------------------------------------------------- > > (Updated May 26, 2016, 6:09 a.m.) > > > Review request for ranger, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, > Pradeep Agrawal, and Velmurugan Periasamy. > > > Bugs: RANGER-985 > https://issues.apache.org/jira/browse/RANGER-985 > > > Repository: ranger > > > Description > ------- > > **Patch Contains** > Along with download excel spreadsheet feature, add support for download file > as CSV format. > On Reports page for searched policies: downloaded policy file should have > multiple policy items on separate row. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > 2f88a9b > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 19a1509 > security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js > c3acf6b > security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js > ecd97e8 > security-admin/src/main/webapp/styles/xa.css f8e7273 > security-admin/src/main/webapp/templates/reports/UserAccessLayout_tmpl.html > 48741ce > security-admin/src/main/webapp/templates/users/UserTableLayout_tmpl.html > f7c90f3 > > Diff: https://reviews.apache.org/r/47448/diff/ > > > Testing > ------- > > Verified : > > 1) CSV download, excel report download. > > 2) Verified format of excel report if single ploicy has multiple policyitems, > it contains multiple rows for multiple policy items of a single policy. > > 3) Verified re-arrangement of columns in HTML report. > > > Thanks, > > Mehul Parikh > >
