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

ENH: Add 'ban user' options to forum moderation #800

Conversation

johnhenley
Copy link
Collaborator

Description of PR...

Adds 'ban user' functionality when reviewing posts for moderation

Changes made

  • Add 'Ban' moderation action to Notifications activity feed
  • Add 'Ban' moderation action to Forums Moderate

How did you test these updates?

local install

image
image

PR Template Checklist

  • Fixes Bug
  • Feature solution
  • Other
  • Requires documentation updates
  • I've updated the documentation already

@johnhenley johnhenley added the enhancement New feature or request label May 10, 2024
@johnhenley johnhenley added this to the 08.01.00 milestone May 10, 2024
@johnhenley johnhenley self-assigned this May 10, 2024
@johnhenley
Copy link
Collaborator Author

@WillStrohl one more for you

Copy link
Contributor

@Timo-Breumelhof Timo-Breumelhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great improvement :-)

@johnhenley
Copy link
Collaborator Author

@WillStrohl this is ready for you to review

Copy link
Member

@WillStrohl WillStrohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@WillStrohl WillStrohl merged commit 1b24b39 into DNNCommunity:dev May 24, 2024
1 check passed
@johnhenley johnhenley deleted the issues/add-ban-user-to-moderator-notification-actions branch May 24, 2024 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants