This PR includes a rejectionReason when rejecting a user's comments as part of a ban with rejectExistingComments = true. This addresses the issue with both spam banning and managing a users ban via the community view.
These changes will impact:
[ ] commenters
[x] moderators
[x] admins
[ ] developers
What changes to the GraphQL/Database Schema does this PR introduce?
None.
Does this PR introduce any new environment variables or feature flags?
No.
If any indexes were added, were they added to INDEXES.md?
n/a
How do I test this PR?
Start the server with cron jobs running
As a user with mod priveledges, navigate to the stream and spam ban a user.
Observe that the ban goes through, and that the users comments are rejected. Additionally, the resulting commentModerationActions should have a reason attached stating that the user was banned.
As a user with mod priveledges, ban a user across all sites via the community view
Observe that the ban goes through, and that the users comments are rejected. Additionally, the resulting commentModerationActions should have a reason attached stating that the user was banned.
As a user with mod priveledges, ban a user across selected sites via the community view
Observe that the ban goes through, and that the users comments on the selected sites are rejected. Additionally, the resulting commentModerationActions should have a reason attached stating that the user was banned.
Where any tests migrated to React Testing Library?
What does this PR do?
This PR includes a rejectionReason when rejecting a user's comments as part of a ban with
rejectExistingComments = true
. This addresses the issue with both spam banning and managing a users ban via the community view.These changes will impact:
What changes to the GraphQL/Database Schema does this PR introduce?
None.
Does this PR introduce any new environment variables or feature flags?
No.
If any indexes were added, were they added to
INDEXES.md
?n/a
How do I test this PR?
Where any tests migrated to React Testing Library?
No.
How do we deploy this PR?
No special considerations should be needed.