search

coralproject / talk

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

[CORL-3104]: Email domain ban confirmation #4560

Closed kabeaty closed 6 months ago

kabeaty commented 6 months ago

What does this PR do?

These changes add a confirmation to email domain bans, so that when selecting this option, the moderator must type "ban" to confirm they want to ban all commenter accounts for an email domain.

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?

As a moderator or admin, go to the Community tab in the admin. In the Status column dropdown for a user, select "Manage Ban". In the ban modal that pops up, see the Email domain ban section. Click the checkbox to Ban all commenter accounts from { domain }. See that Save is now disabled and a warning shows saying this will prevent any commenter from using this email domain and that the user must type in ban to confirm. Once ban is typed into the input, then the Save button should be enabled. Click Save and see that it's applied as expected.

Were any tests migrated to React Testing Library?

How do we deploy this PR?

netlify[bot] commented 6 months ago

Deploy Preview for gallant-galileo-14878c canceled.

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