On 07/01/2015 04:51 PM, Petr Vobornik wrote:
> For those of you who don't want to try the patches:
> * https://pvoborni.fedorapeople.org/images/api-user-show.png
> * https://pvoborni.fedorapeople.org/images/api-user-add.png
> On 07/01/2015 09:35 AM, Martin Kosek wrote:
>> On 06/30/2015 06:35 PM, Petr Vobornik wrote:
>>> First part of API Browser - displaying the metadata in more consumable way.
>>> Second part, how to use it in different languages will be written as wiki
>>> The browser could be later enhanced with more infos and tooltips.
>>> Patch 886 extends backend to send more metadata.
>>> Patch 887,888,889 are webui fixes and prerequisites
>>> Patch 890 is the API browser
>> Thanks, this is a very good start. I looked at a VM with the patches and have
>> couple usability suggestions:
>> 1) It was hard for me to find where the API Browser is. But "IPA Server"
>> as a good tab where it should be though.
> could be moved to "Help" tab when it's introduced. For that we need at least
> one more link.
>> 2) I have strong doubts about the "Objects" tab, this is only understandable
>> users knowledgeable about FreeIPA framework internals. Common API user who
>> want to consume the API and not know about the internals will not know what
>> this is.
>> What I would do is make "API Browser" directly clickable so that it opens the
>> Commands tab. This is what most people will use. Other tabs may be stacked on
>> the left just like with Staged or Deleted users. For now, I would hide
>> as I think it would cause more confusion. If we want to show it, there should
>> be some introduction what it is good for and maybe limitation of showed
>> to only those that has any value for the consumers.
> fixed, there is only "API Browser" and no submenu
>> 3) In Commands tab, we will some more explanatory what the attributes of
>> needs and probably hide some. For example "exclude" is not needed for
> Attributes as follows were kept: label, type, default, default_from, values,
> minlength, maxlength, pattern, minvalue, maxvalue, precision, cli_name,
>> 4) Many attributes have "autofill": True. I wonder how usable it is without
>> knowing the actual default for the attribute. Can we show the default?
> default_from now contains list of attrs which are used for the default value,
> default value created from: givenname, sn
>> 5) I would hide "Output Params" all together given we don't have them set up
>> correctly in FreeIPA framework and they may rather confuse people, with
>> all the HBAC or SUDO with User objects.
> Removed from metadata
>> I may think about it more, there were just my couple first thoughts. Others
>> have different opinions here.
> Other changes:
> * cli options are shown with dashes as in CLI
> * required and multivalued were changed into tags next to option name. 'flags'
> which were shown as the tags are not displayed anymore
> updated patches attached.
I like the new version, good job! ACK from my side.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code