Usermanagement: Password change error display behavior #21239

Closed
michag86 opened this Issue Dec 16, 2015 · 4 comments

Projects

None yet

5 participants

@michag86
Contributor

On changing the password of an user in the user management, the password change error message is shown permanently:
https://github.com/owncloud/core/blob/v8.2.1/settings/js/users/users.js#L657

Is there a better way to show the password change error?

The easiest way is to change "OC.Notification.show" to "OC.Notification.showTemporary".

This notification can be used in password policy app to show, that the password will not comply with the policy. But it is not useful to show this error permanently.

At the moment, the app does not return the correct formatted response. But this is just a small change.

@PVince81 PVince81 added the bug label Dec 17, 2015
@PVince81
Collaborator

@michag86 change it to showTemporary instead.
PR ? 😄

@PVince81 PVince81 added this to the 9.0-current milestone Dec 17, 2015
@michag86
Contributor

@DeepDiver1975 can we backport this to 8.2 and 8.1?

@karlitschek
Member

@PVince81 @DeepDiver1975 are you in favor of a backport?

@DeepDiver1975
Member

@PVince81 @DeepDiver1975 are you in favor of a backport?

no objections - low criticality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment