@sgiehl opened this Pull Request on March 5th 2016 Member

Improves password reset confirmation mail text to inform user that token_auth will change aswell.

New text will be:

Hi %1$s,

A password reset request was received from %2$s. To confirm this password change so you can login with your new credentials, visit the following link:\n\n%3$s

Attention: Changing the password will also change your token_auth. You can look up your new token_auth on your settings page.

Note: this token will expire in 24 hours.

And thank you for using Piwik!

refs #9833

@tsteur commented on March 6th 2016 Owner

Should we maybe also show a warning here?

image

Eg "Changing your password will also change your API token. This token is used for example in the Piwik Mobile app." or maybe we show it instead in success message?

https://github.com/piwik/piwik-ui-tests/blob/master/Login_password_reset_complete.png

@tsteur commented on March 6th 2016 Owner

I'm not sure where to change it and if. Just asking :)

@sgiehl commented on March 6th 2016 Member

Thought about adding it there, but actually the token is changed when clicking on the link in the mail.
Adding it to the form might maybe look like it will be changed when submitting the form.

@tsteur commented on March 6th 2016 Owner

:+1:

@mattab commented on March 15th 2016 Owner

FYI: tweaked the message further (for consistency) in https://github.com/piwik/piwik/commit/409fb2d22670105cfd2652a4987625fc6f19cece

This Pull Request was closed on March 6th 2016
Powered by GitHub Issue Mirror