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 ability for admins to force break a queue and set the mmr multiplier #26

Open
Tracked by #18
mattwells19 opened this issue Mar 9, 2022 · 0 comments · May be fixed by #34
Open
Tracked by #18

Add ability for admins to force break a queue and set the mmr multiplier #26

mattwells19 opened this issue Mar 9, 2022 · 0 comments · May be fixed by #34
Assignees

Comments

@mattwells19
Copy link
Owner

Description

Add admin slash commands with the following abilities:

  • forcebreakq: Removes an active match. Requires a player in a match to be provided to know which match to remove.
  • mmrmult: Updates the MMR multiplier for the current Event.

Acceptance Criteria

  • Only members with the Bot Admin role can successfully execute these commands.
  • All responses are "ephemeral."
  • Issuing the force broken queue command removes the active match that contains the specified player.
    • Reference the kick command on how to require a player to be specified.
  • Issuing the MMR multiplier should accept a parameter for the new multiplier
    • Can be a decimal or whole number
    • Cannot be less than 0 (but can be 0)
  • Entering an invalid MMR multiplier should respond with an appropriate message and not change the MMR multiplier.

Technical Details

The check for the Bot Admin role is already implemented in AdminController so that shouldn't need to change.
Be sure to reference the existing kick and clear admin commands for registering and handling slash commands.
This does not include actually using the MMR multiplier when reporting matches. That will be done as part of a separate issue.

@mattwells19 mattwells19 added this to the Admin Commands milestone Mar 9, 2022
@mattwells19 mattwells19 self-assigned this Mar 11, 2022
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 a pull request may close this issue.

1 participant