-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Improve lost password email #4345
Comments
Mmmm shouldn't it contains the ip? And a link to contact your admin or something? Like if you didn't request it you might want to report that somebody is trying to access your account... |
But do we really want to put that in the body copy? It's a mighty lot of text for a simple password reset. The button should be one of the first elements because it's fine for the majority of cases. That extra info text is for the edgecases where malicious stuff is done, so it can be smaller / grey and further down. |
Hello, is this issue still open? |
This comment has been minimized.
This comment has been minimized.
Superseded by #48095 |
Followup to #4308
Mockup by @Espina2
Comments by @eppfel:
The text was updated successfully, but these errors were encountered: