Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CORL-1255] Move wordlist check after staff approval check #3094

Merged
merged 2 commits into from Aug 10, 2020

Conversation

nick-funk
Copy link
Contributor

What does this PR do?

Move wordlist check after staff approval check.

This allows staff members to comment anything they want, even if it's on the banned word list.

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

None

How do I test this PR?

  • login as staff user
  • go to admin > config, make sure you have some banned words set
  • use the banned words in a comment
  • comment should go through and be posted

This allows staff members to comment anything
they want, even if it's on the banned word list.

CORL-1255
@kgardnr kgardnr added this to the v6.3.2 milestone Aug 10, 2020
@kgardnr kgardnr self-requested a review August 10, 2020 20:06
@kgardnr kgardnr added the 🚀 merge it! Pull requests that should be merged after status checks pass with a review label Aug 10, 2020
@kodiakhq kodiakhq bot merged commit af842e9 into master Aug 10, 2020
@kodiakhq kodiakhq bot deleted the feature/CORL-1255-staff-special branch August 10, 2020 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 merge it! Pull requests that should be merged after status checks pass with a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants