-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Subscriptions: Don't email deleted accounts #93548
Comments
I haven't been able to test this because I don't receive any emails on my subscriber inbox after posting content. Maybe it's my user, or my site. I'm going to leave it and work on another issue. |
I'm taking over this ticket and I'm working on a diff that will prevent new post / comment emails from going out to deleted user accounts. @lsl Do we have a report on follower notification emails going out to deleted user accounts? I can't seem to replicate this. |
Diff: D158732-code |
Approved 👍 |
@TimBroddin would you be interested in giving a try to fix this other issue that is also related to subscribers? I am struggling to reproduce it because it seems to happen only when the subscriber's membership for paid content is renewed. |
Will do, thanks for the review! |
We can close this now that D158732-code is deployed to avoid emailing deleted accounts. The pending part for the count recalculation issue is tracked in #56919 (comment) |
Background: #56919,
This issue is to split off and track a fix for the sending email part, leaving the count recalculation issue as a separate problem.
Done means:
The text was updated successfully, but these errors were encountered: