Skip to content

Conversation

@wescopeland
Copy link
Member

@wescopeland wescopeland commented Nov 30, 2025

This PR adds a few more community email notification filters to bring email rates down another notch.

Now, for forum threads with at least 6 pages (90 comments), if the user is implicitly subscribed, hasn't replied in 90+ days, and isn't the OP, they no longer receive an automatic email.

For threads with 200+ subscribers, implicit subscribers who haven't posted in the last 45 days will no longer receive email notifications. This targets threads where each new comment triggers hundreds of emails. Most normal threads are completely unaffected.

@wescopeland wescopeland requested a review from a team November 30, 2025 00:25
Copy link
Member

@Jamiras Jamiras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've verified the functionality, but I'm not certain I agree with the change.

There are several forum topics that I've participated in where I would want to be notified if they've been necro'd (I don't know how many of them are 6+ pages). I guess I could explicitly subscribe to those, but that's putting the onus on the user for functionality that "just worked" before.

If I get too much spam from a message thread, I just unsubscribe from it. I think the problem we're trying to solve here is exacerbated by the fact that these emails usually go to user's junk mail folders, so it's pointless to be sending them anyway.

I think a better solution would be to move to the daily digest for implicit subscriptions, but I realize that's a bigger project.

@wescopeland wescopeland requested a review from Jamiras December 2, 2025 22:08
@wescopeland
Copy link
Member Author

@Jamiras I've updated the filtering criteria in latest, per the most recent findings.

For threads with 200+ subscribers, implicit subscribers who haven't posted in the last 45 days will no longer receive email notifications. This targets threads where each new comment triggers hundreds of emails. Most normal threads are completely unaffected.

@wescopeland wescopeland merged commit 9460b30 into RetroAchievements:master Dec 2, 2025
13 checks passed
@wescopeland wescopeland deleted the long-thread-email-filter branch December 2, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants