-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
Decouple setting an email for password reset from publishing your threepid to the identity server and support choice of identity server (on registration) #10091
Comments
Setting an e-mail for password reset also allows the enablement of e-mail notifications being sent to it |
I don't think this needs calling out in the UX specifically, since you're not opted into email notifications by default (I just verified this). |
Did you test with a new account or adding an email to your existing account? I’m pretty sure new accounts do have email notifications enabled by default. |
I did test with a new account - I just tested adding an email addres sto an old account and email notifs weren't enabled by default (in this instance at least). Do you have a counter example? |
I guess not. I just tried it with a new account and it worked as you said. Maybe that behavior changed since I created my account because I'm pretty sure it was enabled by default in the past. |
After validating today, latest comps are in Zeplin: https://zpl.io/brMdWo3 |
The material parts of this have moved to other issues, namely: |
The text was updated successfully, but these errors were encountered: