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

Add moderator types #36

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Add moderator types #36

wants to merge 4 commits into from

Conversation

SuperMarioDaBom
Copy link
Member

Resolves #XXX

Changes:

This PR adds the ability for users applying to moderator roles to specify which moderation role they want to apply to. This information is then used to send this application to a channel based upon the role. This is to allow each moderation group to have their own place to discuss & handle applicants. This update is necessary as we continue to work to add proper definitions & separation of concerns for each moderation group.

BandwidthModUpdate-SlashCmd
BandwidthModUpdate-AppMsg

Copy link
Member

@MatthewL246 MatthewL246 left a comment

Choose a reason for hiding this comment

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

General comment: I don't think a Discord bot is the right place for forum mod applications, mainly because forum mods shouldn't necessarily be required to have a Discord account.

Besides that, this seems like a very reasonable change.

@SuperMarioDaBom SuperMarioDaBom marked this pull request as draft December 12, 2024 22:24
@SuperMarioDaBom
Copy link
Member Author

Updated this PR to be a draft, as it seems we have a number of things to figure out before a change like this can be fully deployed (as discussed on Discord).

For now, I will be implementing:

  • When running the mod-application slash command for a forum mod role, instead of popping up the modal, a message will be displayed pointing the user to the proper place to apply on the forum.
  • Only discord mod, vc mod, and forum mod (w/ redirect) will be enabled as we figure out solutions for the other two.

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.

2 participants