-
Notifications
You must be signed in to change notification settings - Fork 0
224 browser auto fill #225
base: staging
Are you sure you want to change the base?
Conversation
Where is the password included in the mail? match4everything/backend/apps/matching/templates/registration/password_reset_email_.html Lines 3 to 14 in 3b5e9d8
|
The new password on |
Sorry, I meant link to reset the password.
I tested that and it updated fine here. (Tested on Windows&Chrome) with 64d4b5b |
@Baschdl could you answer the above? |
One more idea: This might not work on non-ssl pages, I tested with a local SSL Proxy and a properly signed certificate |
I used Firefox on Mac OS |
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.
Saving of username+password works, auto-generation of password doesn't work in Safari, Firefox and Chrome on Mac
Closes #224
Introduces a new hidden (via css) field in the password reset form with the username and appropriate autocomplete settings so that credentials will be saved in the browser.
Considered if this was bad practice but considering that our password reset mail includes the e-mail as well as the password this should be good. Additionally with this changes users will be able to use autogenerated passwords as suggested by current browsers which also benefits account security
ToDo: