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

Subscriptions: Don't email deleted accounts #93548

Closed
2 tasks done
lsl opened this issue Aug 15, 2024 · 7 comments
Closed
2 tasks done

Subscriptions: Don't email deleted accounts #93548

lsl opened this issue Aug 15, 2024 · 7 comments
Assignees
Labels
Groundskeeping Issues handled through Dotcom Groundskeeping rotations [Pri] High Address as soon as possible after BLOCKER issues

Comments

@lsl
Copy link
Contributor

lsl commented Aug 15, 2024

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:

  • When sending new post / follower notification emails to subscribers we need to check the receiving account hasn't been deleted.
  • If the account has been deleted, don't send the email.
@lsl lsl converted this from a draft issue Aug 15, 2024
@lsl lsl added the [Pri] High Address as soon as possible after BLOCKER issues label Aug 15, 2024
@miksansegundo miksansegundo self-assigned this Aug 15, 2024
@miksansegundo miksansegundo moved this from Ready for Development to In Progress in Automattic Prioritization: The One Board ™ Aug 15, 2024
@miksansegundo
Copy link
Contributor

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.

@miksansegundo miksansegundo removed their assignment Aug 15, 2024
@miksansegundo miksansegundo moved this from In Progress to Ready for Development in Automattic Prioritization: The One Board ™ Aug 15, 2024
@TimBroddin TimBroddin self-assigned this Aug 19, 2024
@TimBroddin TimBroddin moved this from Ready for Development to In Progress in Automattic Prioritization: The One Board ™ Aug 19, 2024
@TimBroddin
Copy link
Contributor

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.

@TimBroddin
Copy link
Contributor

Diff: D158732-code

@TimBroddin TimBroddin moved this from In Progress to Needs Review in Automattic Prioritization: The One Board ™ Aug 19, 2024
@miksansegundo
Copy link
Contributor

Approved 👍

@miksansegundo
Copy link
Contributor

@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.

@TimBroddin
Copy link
Contributor

Will do, thanks for the review!

@TimBroddin TimBroddin moved this from Needs Review to Done 🎉 in Automattic Prioritization: The One Board ™ Aug 20, 2024
@TimBroddin TimBroddin added the Groundskeeping Issues handled through Dotcom Groundskeeping rotations label Aug 23, 2024
@miksansegundo
Copy link
Contributor

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Groundskeeping Issues handled through Dotcom Groundskeeping rotations [Pri] High Address as soon as possible after BLOCKER issues
Development

No branches or pull requests

3 participants