Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-14 Thread Rob Crittenden
Dmitri Pal wrote: In addition to the issues I explain above here is what I also noticed: 1) As we mentioned there is no Description in ACI. The description and name is the same field for ACI. Description is in the Meta data, and gets returned with ipa permission_show, role_show, and

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-14 Thread Rob Crittenden
Dmitri Pal wrote: Adam Young wrote: On 12/13/2010 11:27 AM, Dmitri Pal wrote: Sorry this whole part just does not make sense to me. What is the target group? Where it came from? One ACI that uses this is 'add_user_to_default_group. This is used in the permission 'useradmin'. The json

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Dmitri Pal
Dmitri Pal wrote: Adam Young wrote: Dmitri, While I don't expect you to do the review of the patch, I would appreciate at least a visual inspection of the completed UI. Since there seems to be something wrong with the install/UI right now, I've posed the lates on my Fedora People page.

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Adam Young
Lets walk through it tomorrow when I am in the office. On 12/13/2010 11:05 AM, Dmitri Pal wrote: Adam Young wrote: Dmitri, While I don't expect you to do the review of the patch, I would appreciate at least a visual inspection of the completed UI. Since there seems to be something

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Adam Young
On 12/13/2010 11:05 AM, Dmitri Pal wrote: Adam Young wrote: Dmitri, While I don't expect you to do the review of the patch, I would appreciate at least a visual inspection of the completed UI. Since there seems to be something wrong with the install/UI right now, I've posed the lates on

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Dmitri Pal
In addition to the issues I explain above here is what I also noticed: 1) As we mentioned there is no Description in ACI. The description and name is the same field for ACI. Description is in the Meta data, and gets returned with ipa permission_show, role_show, and privilege_show May

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Adam Young
On 12/13/2010 01:28 PM, Dmitri Pal wrote: In addition to the issues I explain above here is what I also noticed: 1) As we mentioned there is no Description in ACI. The description and name is the same field for ACI. Description is in the Meta data, and gets returned with ipa

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Adam Young
On 12/13/2010 11:27 AM, Dmitri Pal wrote: Sorry this whole part just does not make sense to me. What is the target group? Where it came from? One ACI that uses this is 'add_user_to_default_group. This is used in the permission 'useradmin'. The json response for permission-show

Re: [Freeipa-devel] ACI permissions UI up for review

2010-12-13 Thread Dmitri Pal
Adam Young wrote: On 12/13/2010 11:27 AM, Dmitri Pal wrote: Sorry this whole part just does not make sense to me. What is the target group? Where it came from? One ACI that uses this is 'add_user_to_default_group. This is used in the permission 'useradmin'. The json response for

[Freeipa-devel] ACI permissions UI up for review

2010-12-11 Thread Adam Young
Dmitri, While I don't expect you to do the review of the patch, I would appreciate at least a visual inspection of the completed UI. Since there seems to be something wrong with the install/UI right now, I've posed the lates on my Fedora People page. You should be able to see it from here: