Skip to content
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

Wrong message when admin resets password for a user #852

Closed
malle-pietje opened this issue Feb 15, 2018 · 2 comments
Closed

Wrong message when admin resets password for a user #852

malle-pietje opened this issue Feb 15, 2018 · 2 comments
Labels
up-for-grabs Not assigned yet
Milestone

Comments

@malle-pietje
Copy link

When an admin resets a password for a user (who has not yet confirmed registration, open registration is disabled) using the "Send the user a link that will allow them to choose their own password" option, the message the admin sees is:
If the email matches an account in our system, a password reset link will be sent to .

No email message is sent to the user's email address which is correct behavior IMHO.

However, I think the message should inform the admin this only applies to users who have set their password or for activated accounts.

@lcharette lcharette added the up-for-grabs Not assigned yet label Jun 27, 2019
@amosfolz
Copy link
Contributor

amosfolz commented Jul 8, 2019

Should be a simple fix. The message at this line needs to be updated, probably with two new translation sub keys.

However, I think the message should inform the admin this only applies to users who have set their password or for activated accounts.

Since this is an access-controlled resource I think it would be safe to even perform a check and then return a different message depending on the outcome.

I will come back to this once #1006 is merged.

@lcharette
Copy link
Member

Messages have been updated :

Old :
Capture d’écran, le 2023-12-10 à 21 08 14

New:
Capture d’écran, le 2023-12-10 à 21 24 42

@lcharette lcharette moved this from Done (5.1) to Done in UserFrosting Task Planner Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
up-for-grabs Not assigned yet
Projects
Status: Done
Development

No branches or pull requests

3 participants