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

Review email notification wording #185

Closed
brynwhyman opened this issue Jun 11, 2019 · 5 comments
Closed

Review email notification wording #185

brynwhyman opened this issue Jun 11, 2019 · 5 comments

Comments

@brynwhyman
Copy link

The email wording hasn't been reviewed outside of the development team.

Raising this to review the wording. See templates here: https://github.com/silverstripe/silverstripe-mfa/tree/master/templates/SilverStripe/MFA/Email

@brynwhyman
Copy link
Author

cc @silverstripeux

@newleeland
Copy link

newleeland commented Jun 19, 2019

Its looking good.
We could consider to add an additional line at the end of AccountReset.ss
If you didn't request an account reset, feel free to delete this email and carry on with your day.
Just in case an admin sends a request by accident (edge case)

@sachajudd
Copy link
Contributor

Some other suggestions: "If you did not request a reset, you can ignore this message", "If you didn’t make this request, please ignore this email.", or "If you didn't request a reset password, you can safely delete this email." I'm not sure about including "and carry on with your day." in the email 🤔

@brynwhyman
Copy link
Author

I like the look of: "If you didn’t make this request, please ignore this email." because it could be added to the standard password reset email template - which is also missing a similar sentence.

Although, I'd say "did not" would be more clear than "didn't"

@brynwhyman
Copy link
Author

I've raised the request separately, here: #214

Still happy for that issue to be updated if needed, but closing this issue. Thanks for reviewing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants