-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix / old password form error after validation #166
Fix / old password form error after validation #166
Conversation
You're now trying to match the filled in old password with the new password. That's never what we want, right? The user wants to change it. |
Good point regarding the api error. I looked at the way it is currently done with the validation between the 2 new passwords. I will go back to the drawing board! Update: discussed with Roy that we do need the translations in order to display the error in Spanish as well. We do now however take the 'old password does not match' api error in account and show it accordingly! |
23fd8af
to
739aa11
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, provided you undo the "passwords_do_not_match" update, and put its value into "old_password_does_not_match".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Interesting to see that we missed the autocomplete
attribute previously for the password field, but these thing can happen ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
This PR
autocomplete
attribute to improve accessibility