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

[WarnSystem] Replace role mute system, with Discord built-in timeouts #217

Open
wants to merge 11 commits into
base: v3
Choose a base branch
from

Conversation

Luc1412
Copy link

@Luc1412 Luc1412 commented Apr 3, 2024

Pull request type

This PR replaces the old role-based mute system, with Discord's timeouts.

Description of the changes

  • Duration for warns with level 2 are now mandatory and are limited to 28d
  • Removed everything related to role management and role settings for mutes
  • Mutes got renamed to timeouts
  • Warn System doesn't require role management related permissions, instead it requires moderate members permission

Check-list

If useful, make a bullet list of the things done or not (will show up in the PR list). Example:

  • Properly naming command to remove timeouts
  • Require time for warn level 2 and check if it's within 28d in API
  • Should we handle unmutes? If so, how?
  • Testing

@Luc1412 Luc1412 marked this pull request as ready for review April 4, 2024 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant