On 02/19/2013 02:08 PM, Sumit Bose wrote:
On Tue, Feb 19, 2013 at 02:01:24PM +0100, Petr Vobornik wrote:
On 02/19/2013 01:40 PM, Sumit Bose wrote:
On Thu, Feb 14, 2013 at 09:56:44AM -0600, Endi Sukma Dewata wrote:
On 2/14/2013 6:30 AM, Petr Vobornik wrote:
If they are mutually exclusive, they probably should be separated using
radio buttons like this:

   PAC: ( ) None
        (o) Type:
            [x] MS-PAC
            [ ] PAD

You missed one option: nothing selected. It can be solved by adding   '(
) Inherited' radio.

I wouldn't have guessed that :) I agree we should add the
'Inherited' option.


Would it be possible to print the inherited values here, i.e. the
default values given in the ipakrbauthzdata attribute of ipaGuiConfig
object? I think this would improve the user experience because you do
not have to remember/look up the default values.

bye,
Sumit


It is possible, but not straightforward.

I see two options:
   1) Currently Web UI loads config at start so we can use it. The
disadvantage is that the value might be changed and so the displayed
value might be incorrect.
   2) Other option is to run config-show along with service-show in a
batch to get the up-to-date value.

I'm not a big fan of #2 but it is probably better solution.
I agree with both :-). Since it is not straightforward I guess it would
be better not to add this with this ticket but open a new one. Do you
agree?

Yes


bye,
Sumit

--
Petr Vobornik


--
Petr Vobornik

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to