[Freeipa-devel] [PATCH] 133 Action list for user password

[Petr Vobornik]

Just for record: this patch is replaced by my patches 141, 142 (sent to 
list on May 24).

On 05/14/2012 07:12 PM, Endi Sukma Dewata wrote:
> On 5/3/2012 3:59 AM, Petr Vobornik wrote:
>> Currently the user password is shown as follows in the details page:
>> Password: Reset Password
>>
>> This is inconsistent with the rest of the page because the 'Reset
>> Password' is an action, not the value of the password.
>>
>> Now password is shown as follows:
>> Password: ******* (if set)
>> Password: (if not set)
>>
>> Reset password action was moved to Action List.
>>
>> https://fedorahosted.org/freeipa/ticket/2248
>>
>> Note: I will address enabling/disabling 'reset password' option in
>> action list in ticket #2318.
>
> Just for the record:
>
> On 5/11/2012 11:36 AM, Petr Vobornik wrote:
>  > On 05/08/2012 01:47 AM, Endi Sukma Dewata wrote:
>  >> 10. The 'Action List' in ticket #2248 for the password reset is
> actually
>  >> different than the action list for Enable/Disable. I was proposing to
>  >> create a small panel under the 'Account Settings' section, probably
>  >> something like this:
>  >>
>  >> ---------------------------------------------------------------
>  >> Account Settings
>  >> +-----------------+
>  >> User login: admin | Actions: |
>  >> Password: | Reset password |
>  >> Password expiration: +-----------------+
>  >> UID:
>  >> GID:
>  >>
>  >> This panel might better be called 'Action Panel' to distinguish from
> the
>  >> dropdown 'Action List' on the top. The reason for this panel is that a
>  >> Password Reset only affects an aspect of the user, not the entire
> object
>  >> like Enable/Disable (although that can also be argued differently), so
>  >> the action should be placed where it's relevant, in this case near the
>  >> Password field. The same concept will be used for ticket #2250, #2251,
>  >> #2252.
>  >>
>  >
>  > I'll consider my patch #133 NACKed and first create the action panels.
>


-- 
Petr Vobornik