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

FIX: exclude suppressed category topics in digest even if unmuted. #14793

Merged
merged 1 commit into from Nov 3, 2021

Conversation

vinothkannans
Copy link
Member

Previously, suppressed category topics are included in the digest emails if the user visited that topic before, and the TopicUser record is created with any notification level except 'muted'.

Previously, suppressed category topics are included in the digest emails if the user visited that topic before and the `TopicUser` record is created with any notification level except 'muted'.
@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/suppressed-category-included-in-digest/203304/6

@tgxworld tgxworld added the 👍 OP to merge PR author can go ahead and merge! label Nov 3, 2021
@vinothkannans vinothkannans merged commit 53b7220 into main Nov 3, 2021
@vinothkannans vinothkannans deleted the suppress-category-digest-fix branch November 3, 2021 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👍 OP to merge PR author can go ahead and merge!
4 participants