search

coralproject / talk

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

fix: check for dsa enabled when sending rejection notifications #4584

Closed kabeaty closed 5 months ago

kabeaty commented 5 months ago

What does this PR do?

Only send rejected comment notifications if DSA is enabled.

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?

Reject comments by a user with DSA enabled and not enabled. See that when enabled, you get a notification as that user. When disabled, you should not get a notification as that user.

Were any tests migrated to React Testing Library?

How do we deploy this PR?

netlify[bot] commented 5 months ago

Deploy Preview for gallant-galileo-14878c canceled.

Name Link
Latest commit f2cf1be79661545832a531e69617d153dab5d017
Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/65fdac9b875ea60008ed0216