Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-12-06 Thread Petr Vobornik
On 12/06/2011 06:38 AM, Endi Sukma Dewata wrote: On 11/18/2011 12:27 PM, Endi Sukma Dewata wrote: Now the methods metadata seem to be a subset of commands metadata, so we probably can change the UI to use commands metadata and not pull the methods metadata. I did this change in the updated

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-12-06 Thread Rob Crittenden
Petr Vobornik wrote: On 12/06/2011 06:38 AM, Endi Sukma Dewata wrote: On 11/18/2011 12:27 PM, Endi Sukma Dewata wrote: Now the methods metadata seem to be a subset of commands metadata, so we probably can change the UI to use commands metadata and not pull the methods metadata. I did this

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-12-06 Thread Rob Crittenden
Endi Sukma Dewata wrote: On 12/6/2011 10:30 AM, Rob Crittenden wrote: Updated patch attached. Web UI - ACK. Server side - seems fine - I would give it an ACK, but I'm not sure if I'm the right person for it. I think this is too radical a change. We can only bump the minor version in this

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-12-06 Thread Endi Sukma Dewata
On 12/6/2011 4:02 PM, Rob Crittenden wrote: Web UI - ACK. Server side - seems fine - I would give it an ACK, but I'm not sure if I'm the right person for it. I think this is too radical a change. We can only bump the minor version in this release so while the api can be modified it still needs

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Petr Vobornik
On 11/17/2011 08:27 PM, Endi Sukma Dewata wrote: The json_metadata command has been modified to return the command metadata. The API.txt has been updated as well. Needed by ticket #388 ACK from UI perspective. Ipalib changes seems fine to me, but I'm not 100% sure. Note: this patch

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Alexander Bokovoy
On Thu, 17 Nov 2011, Endi Sukma Dewata wrote: The json_metadata command has been modified to return the command metadata. The API.txt has been updated as well. Needed by ticket #388 ... --- a/ipalib/plugins/internal.py +++ b/ipalib/plugins/internal.py @@ -49,15 +52,25 @@ class

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Endi Sukma Dewata
On 11/18/2011 4:58 AM, Petr Vobornik wrote: On 11/17/2011 08:27 PM, Endi Sukma Dewata wrote: The json_metadata command has been modified to return the command metadata. The API.txt has been updated as well. Needed by ticket #388 ACK from UI perspective. Ipalib changes seems fine to me, but