coralproject / talk

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

DSA fix: only check for duplicate report if not anonymous user #4486

Closed kabeaty closed 8 months ago

kabeaty commented 8 months ago

What does this PR do?

Update so that we only check for duplicate DSA reports submitted by a user for a particular comment if the user is signed in and it is not an anonymous report.

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?

Make sure you're signed out. Try to DSA report the same comment twice. See that there are no errors and both show in the DSA Reports tab.

Were any tests migrated to React Testing Library?

How do we deploy this PR?

netlify[bot] commented 8 months ago

Deploy Preview for gallant-galileo-14878c ready!

Name Link
Latest commit 0b33291e6feeeac6db82c72e5365bb631185a6e3
Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/65a813a13769e30008374532
Deploy Preview https://deploy-preview-4486--gallant-galileo-14878c.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.