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

//shuffle command wipe the queue #235

Closed
xadll opened this issue Jan 5, 2024 · 3 comments · Fixed by #243
Closed

//shuffle command wipe the queue #235

xadll opened this issue Jan 5, 2024 · 3 comments · Fixed by #243
Labels
documentation Improvements or additions to documentation

Comments

@xadll
Copy link

xadll commented Jan 5, 2024

No description provided.

@wsrvn
Copy link
Collaborator

wsrvn commented Jan 5, 2024

it is assumed you dont care about the queue if you do shuffle

@wsrvn wsrvn added the invalid This doesn't seem right label Jan 5, 2024
@xadll
Copy link
Author

xadll commented Jan 5, 2024

it is assumed you dont care about the queue if you do shuffle

I thought that, according to the description, the command shuffles positions of the queued maps in the queue (jukebox).
OK, then the help is misleading.

help: `Shuffle the map queue.`

@xadll
Copy link
Author

xadll commented Jan 5, 2024

it is assumed you dont care about the queue if you do shuffle

I thought that, according to the description, the command shuffles positions of the queued maps in the queue (jukebox). OK, then the help is misleading.

help: `Shuffle the map queue.`

So, I recommend changing the help description to Shuffle the map list.
And also move the help command to MapCommands category, it is currently in the JukeboxCommands one.

@wsrvn wsrvn added documentation Improvements or additions to documentation and removed invalid This doesn't seem right labels Jan 6, 2024
@wsrvn wsrvn linked a pull request Jan 14, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants