You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Richard Zang <rz...@hortonworks.com> on 2015/12/15 20:09:17 UTC

Review Request 41388: Role-Based Access Control: UX and Bugs

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

Review request for Ambari and Xi Wang.


Bugs: AMBARI-14381
    https://issues.apache.org/jira/browse/AMBARI-14381


Repository: ambari


Description
-------

Role-Based Access Control: UX and Bugs


Diffs
-----

  ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/ClustersManageAccessCtrl.js 1f3d3f6 
  ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js 4799e93 
  ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/groups/GroupsEditCtrl.js 9e0c220 
  ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/users/UsersShowCtrl.js cd4e020 
  ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html 64369ab 
  ambari-admin/src/main/resources/ui/admin-web/app/views/groups/edit.html 4934e1e 
  ambari-admin/src/main/resources/ui/admin-web/app/views/leftNavbar.html 7ff564a 
  ambari-admin/src/main/resources/ui/admin-web/app/views/users/list.html f2da12a 
  ambari-admin/src/main/resources/ui/admin-web/app/views/users/show.html ca89e39 
  ambari-web/app/controllers/global/user_settings_controller.js 8a50059 
  ambari-web/app/templates/common/settings.hbs 899b9cf 

Diff: https://reviews.apache.org/r/41388/diff/


Testing
-------

Manually tested on live cluster. All unit tests passed


Thanks,

Richard Zang

Re: Review Request 41388: Role-Based Access Control: UX and Bugs

Posted by Xi Wang <xw...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41388/#review110546
-----------------------------------------------------------

Ship it!


Ship It!

- Xi Wang


On Dec. 15, 2015, 11:09 a.m., Richard Zang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41388/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2015, 11:09 a.m.)
> 
> 
> Review request for Ambari and Xi Wang.
> 
> 
> Bugs: AMBARI-14381
>     https://issues.apache.org/jira/browse/AMBARI-14381
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Role-Based Access Control: UX and Bugs
> 
> 
> Diffs
> -----
> 
>   ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/ClustersManageAccessCtrl.js 1f3d3f6 
>   ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js 4799e93 
>   ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/groups/GroupsEditCtrl.js 9e0c220 
>   ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/users/UsersShowCtrl.js cd4e020 
>   ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html 64369ab 
>   ambari-admin/src/main/resources/ui/admin-web/app/views/groups/edit.html 4934e1e 
>   ambari-admin/src/main/resources/ui/admin-web/app/views/leftNavbar.html 7ff564a 
>   ambari-admin/src/main/resources/ui/admin-web/app/views/users/list.html f2da12a 
>   ambari-admin/src/main/resources/ui/admin-web/app/views/users/show.html ca89e39 
>   ambari-web/app/controllers/global/user_settings_controller.js 8a50059 
>   ambari-web/app/templates/common/settings.hbs 899b9cf 
> 
> Diff: https://reviews.apache.org/r/41388/diff/
> 
> 
> Testing
> -------
> 
> Manually tested on live cluster. All unit tests passed
> 
> 
> Thanks,
> 
> Richard Zang
> 
>