-
Notifications
You must be signed in to change notification settings - Fork 5
Make it possible for logged in users to update email and password #525
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
Conversation
✅ Deploy Preview for ami-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ami-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@annavik there was one required setting missing in the Django Djoser settings (https://djoser.readthedocs.io/en/latest/settings.html#password-reset-confirm-url). I updated that and I am able to reset my password! The one thing I noticed is that I couldn't use my password manager with the form fields. When I try to make a selection the focus jumps to the next field. |
@mihow I added an issue about the password manager thing, I suspect could be a similar situation for password reset, at least worth double checking. Marking this one as ready for review so we can get it out of the way! :) |
In this PR we make it possible for logged in users to update email and password. Before you had to contact an administrator to do so.
Some issues to look into:
User flow:
We show email and password as static values, but with edit buttons visible.

On edit button click, a mini form shows up. The mini form will have its own isolated state. This is because we are making special backend calls on email/password update, outside the user info update.
