coralproject / talk

A better commenting experience from Vox Media
https://coralproject.net
Other
1.89k stars 355 forks source link

[CORL-2976]: Notifications for comment rejection updates #4443

Closed kabeaty closed 9 months ago

kabeaty commented 9 months ago

What does this PR do?

These changes update the copy sent out for notifications when comments are rejected to be more friendly and descriptive. It also displays the custom reason entered for the Other rejection reason now. And comments rejected for banned words do not send out a notification.

These changes will impact:

What changes to the GraphQL/Database Schema does this PR introduce?

Adds in the customReason introduced in another PR so that it can be displayed here if available.

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?

You can test this by making a bunch of comments and rejecting one comment for each rejection reason. Check in the notifications for the user who created those comments that you rejected. See the updated text displayed.

Where any tests migrated to React Testing Library?

How do we deploy this PR?

netlify[bot] commented 9 months ago

Deploy Preview for gallant-galileo-14878c canceled.

Name Link
Latest commit e7438f9d7b4c22fde88eaa13ac62f525169946f5
Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/6569ebe087f4050008261eb3