-
Notifications
You must be signed in to change notification settings - Fork 0
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 Support For A Queue Role #14
Comments
Good call - will require support for admins closing the queue via the bot. Making an issue for that now that will need to be addressed before this one. |
I think we could either do this or just have cypher @ the people in the specific queue, since we are planning to have multiple queues at some point |
That's true, although I think having the bot manage some queue roles would be nicer from a user perspective. Seeing all of those ats is going to be ugly and it'll be very obvious to the users what queue they're getting pinged for if they can literally see something along the lines of |
Actually there is an added benefit of creating multiple queue roles in that you can use them in discord for the duration of the queue. It is going to be harder to write to ensure they are cleaned up smoothly, bear in mind. |
To avoid unnecessary @'ing for all 10mans players (or manually going through the queue and @'ing people), there should be support for a dynamic queue role for those added in a queue. @'ing the role only @'s people in the queue. Adding up assigns the role, and it can be removed on queue empty. This can also be extended to only @ people not in voice channel(s).
The text was updated successfully, but these errors were encountered: