You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ambari.apache.org by Andriy Babiichuk <ab...@hortonworks.com> on 2014/10/15 13:33:27 UTC

Review Request 26737: Change the way UI requests privileges of logged user

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

Review request for Ambari, Aleksandr Kovalenko and Oleg Nechiporenko.


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


Repository: ambari


Description
-------

UI should request privileges of currently logged user by using url 
{code}
http://c6401.ambari.apache.org:8080/api/v1/users/admin?fields=*,privileges/PrivilegeInfo/*
{code}
(replace * with the fields used in UI)


Diffs
-----

  ambari-web/app/assets/test/tests.js 74f8a4c 
  ambari-web/app/mappers/users_mapper.js 00ac6c3 
  ambari-web/app/router.js b347512 
  ambari-web/app/utils/ajax/ajax.js 48def9c 
  ambari-web/test/router_test.js PRE-CREATION 

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


Testing
-------

1 test added 
4583 tests complete (6 seconds)
60 tests pending


Thanks,

Andriy Babiichuk

Re: Review Request 26737: Change the way UI requests privileges of logged user

Posted by Aleksandr Kovalenko <ak...@hortonworks.com>.

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

Ship it!


Ship It!

- Aleksandr Kovalenko


On Жов. 15, 2014, 11:35 до полудня, Andriy Babiichuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26737/
> -----------------------------------------------------------
> 
> (Updated Жов. 15, 2014, 11:35 до полудня)
> 
> 
> Review request for Ambari, Aleksandr Kovalenko and Oleg Nechiporenko.
> 
> 
> Bugs: AMBARI-7771
>     https://issues.apache.org/jira/browse/AMBARI-7771
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> UI should request privileges of currently logged user by using url 
> http://c6401.ambari.apache.org:8080/api/v1/users/admin?fields=*,privileges/PrivilegeInfo/*
> (replace * with the fields used in UI)
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/assets/test/tests.js 74f8a4c 
>   ambari-web/app/mappers/users_mapper.js 00ac6c3 
>   ambari-web/app/router.js b347512 
>   ambari-web/app/utils/ajax/ajax.js 48def9c 
>   ambari-web/test/router_test.js PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/26737/diff/
> 
> 
> Testing
> -------
> 
> 1 test added 
> 4583 tests complete (6 seconds)
> 60 tests pending
> 
> 
> Thanks,
> 
> Andriy Babiichuk
> 
>

Re: Review Request 26737: Change the way UI requests privileges of logged user

Posted by Andriy Babiichuk <ab...@hortonworks.com>.

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

(Updated Окт. 15, 2014, 11:35 д.п.)


Review request for Ambari, Aleksandr Kovalenko and Oleg Nechiporenko.


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


Repository: ambari


Description (updated)
-------

UI should request privileges of currently logged user by using url 
http://c6401.ambari.apache.org:8080/api/v1/users/admin?fields=*,privileges/PrivilegeInfo/*
(replace * with the fields used in UI)


Diffs
-----

  ambari-web/app/assets/test/tests.js 74f8a4c 
  ambari-web/app/mappers/users_mapper.js 00ac6c3 
  ambari-web/app/router.js b347512 
  ambari-web/app/utils/ajax/ajax.js 48def9c 
  ambari-web/test/router_test.js PRE-CREATION 

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


Testing
-------

1 test added 
4583 tests complete (6 seconds)
60 tests pending


Thanks,

Andriy Babiichuk

Re: Review Request 26737: Change the way UI requests privileges of logged user

Posted by Oleg Nechiporenko <on...@apache.org>.

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

Ship it!


Ship It!

- Oleg Nechiporenko


On Oct. 15, 2014, 11:33 a.m., Andriy Babiichuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26737/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2014, 11:33 a.m.)
> 
> 
> Review request for Ambari, Aleksandr Kovalenko and Oleg Nechiporenko.
> 
> 
> Bugs: AMBARI-7771
>     https://issues.apache.org/jira/browse/AMBARI-7771
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> UI should request privileges of currently logged user by using url 
> {code}
> http://c6401.ambari.apache.org:8080/api/v1/users/admin?fields=*,privileges/PrivilegeInfo/*
> {code}
> (replace * with the fields used in UI)
> 
> 
> Diffs
> -----
> 
>   ambari-web/app/assets/test/tests.js 74f8a4c 
>   ambari-web/app/mappers/users_mapper.js 00ac6c3 
>   ambari-web/app/router.js b347512 
>   ambari-web/app/utils/ajax/ajax.js 48def9c 
>   ambari-web/test/router_test.js PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/26737/diff/
> 
> 
> Testing
> -------
> 
> 1 test added 
> 4583 tests complete (6 seconds)
> 60 tests pending
> 
> 
> Thanks,
> 
> Andriy Babiichuk
> 
>