-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add comment moderation feature #531
Conversation
@kirtangajjar is this up for review or still in progress? |
@jeffpaul The PR is ready, I just have to update the description. Once it's done, then ready to review. |
@Sidsector9 Done. Fixed the PHPCS issues and resolved your feedback as well :) |
@Sidsector9 this might be a good PR to pick up (or perhaps close and open a subsequent related one) to use as an example of adding a new feature with the v3 release; thoughts? |
Since we're adding this to core, it would be a good example for internal developers only. Third-party devs will still need to use hooks that will ship out with v3. |
@Sidsector9 sorry, yeah, I meant to effectively have v3 stable in |
@Sidsector9 I've migrated all code changes here into the new Feature-first format. I still need to add E2E tests for this new feature but it is ready for another review when you have some time. Edit: tests are there now so this should be good to go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR and making the changes. Tested and works well 👍
Noting for anyone who comes here looking for more context that comments will be sent into moderation if they're flagged by OpenAI Moderation for the following categories: https://platform.openai.com/docs/guides/moderation/overview |
Description of the Change
Adds comment moderation feature.
Closes #331
How to test the Change
Changelog Entry
Credits
Props @kirtangajjar