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

Remove the AdminOnly check for the /remove command. #5

Open
BlitzOffline opened this issue Feb 12, 2022 · 0 comments
Open

Remove the AdminOnly check for the /remove command. #5

BlitzOffline opened this issue Feb 12, 2022 · 0 comments
Assignees

Comments

@BlitzOffline
Copy link
Owner

Currently the /remove command is admin only but it should be accessible to others as well. I am not really sure how it should work tho.
I have a few ideas tho:

  • Leave it as is
  • Make it vote maybe? Tho idk how that would work when someone uses /remove 5 which removes the next 5 songs. (maybe with reactions)
  • Make it take in an index again instead of the amount and allow people to only remove the songs requested by themselves (admins should be able to remove anyone's songs). Also might want to allow everyone to remove songs added by the bot with the radio feature or maybe do something as the second option so a vote with reactions?
@BlitzOffline BlitzOffline self-assigned this Nov 14, 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

No branches or pull requests

1 participant