-
Notifications
You must be signed in to change notification settings - Fork 4k
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 displayed article comment counts (take into account comments hidden after the author is assigned spam
)
#20621
Comments
Thanks for the issue, we will take it into consideration! Our team of engineers is busy working on many types of features, please give us time to get back to you. To our amazing contributors: issues labeled If this is a feature request from an external contributor (not core team at Forem), please close the issue and re-post via GitHub Discussions. To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please ping @forem-team. The OSS Community Manager or the engineers on OSS rotation will follow up. For full info on how to contribute, please check out our contributors guide. |
spam
spam
spam
spam
)
I think, the count in the "View full discussion (n comments)" was not updated, so may show wrong number in some cases. |
Describe the bug
Article comment counts are not updating when comment authors are being applied with the
spam
role.To Reproduce
spam
role to one of the comment authorsExpected behavior
The comment count should show the number of visible comments on the article (i.e. not counting those from users applied with
spam
).Screenshots
Additional context
The text was updated successfully, but these errors were encountered: