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

Email notification when adding/removing email address. #3317

Open
ewdurbin opened this issue Mar 20, 2018 · 5 comments
Open

Email notification when adding/removing email address. #3317

ewdurbin opened this issue Mar 20, 2018 · 5 comments
Labels
email Related to emails feature request needs discussion a product management/policy issue maintainers and users should discuss UX/UI design, user experience, user interface

Comments

@ewdurbin
Copy link
Member

ewdurbin commented Mar 20, 2018

We should notify all email addresses on an account when an email address is added/removed..

We currently only notify for primary email changes. It's probably worth letting the Primary Email address know what's going on.

@waseem18
Copy link
Contributor

waseem18 commented Mar 20, 2018

We should also probably have a Manage Notifications page as this can generate sizable email traffic - considering similar Email feature #997

@di
Copy link
Member

di commented Mar 20, 2018

I'm on the fence about this. A user can't do anything with an added email address until it's made primary, so anyone who's account is compromised will likely receive both notifications anyways.

Like @waseem18 said, for uncompromised users, the action of just setting a new primary email address would generate a lot of emails.

@brainwane brainwane added needs UX/UI review needs discussion a product management/policy issue maintainers and users should discuss feature request labels Mar 21, 2018
@brainwane brainwane added this to the 6. Post Legacy Shutdown milestone Apr 2, 2018
@brainwane brainwane added the blocked Issues we can't or shouldn't get to yet label May 16, 2019
@brainwane
Copy link
Contributor

We still need to make a policy decision about this. But if we decide to do this, then to do this the right way, we should wait till we have #5863 implemented, so we can draw on the event logging and use it to trigger this notification.

@nlhkabu nlhkabu added UX/UI design, user experience, user interface and removed UX/UI design, user experience, user interface needs UX/UI review labels May 20, 2019
@brainwane brainwane removed the blocked Issues we can't or shouldn't get to yet label Aug 15, 2019
@brainwane
Copy link
Contributor

Now unblocked, but we do need to discuss approach - @woodruffw ?

@woodruffw
Copy link
Member

Thanks for the ping!

I believe that either @ewdurbin or @dstufft mentioned that SQLAlchemy has a notion of triggers/events -- one potential approach would be to leverage those on creation of the relevant email event logs, and then dispatch emails from there.

@miketheman miketheman added the email Related to emails label Oct 8, 2024
@miketheman miketheman mentioned this issue Oct 8, 2024
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
email Related to emails feature request needs discussion a product management/policy issue maintainers and users should discuss UX/UI design, user experience, user interface
Projects
None yet
Development

No branches or pull requests

7 participants