search

lythx / trakman

Trackmania Forever server controller written in TypeScript
https://trakman.ptrk.eu
MIT License
24 stars 4 forks source link

//shuffle command wipe the queue #235

Closed xadll closed 7 months ago

wsrvn commented 7 months ago

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

xadll commented 7 months ago

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.

https://github.com/lythx/trakman/blob/fc8d4b0bdbdf74464eaeb0f6638bf9c3c171789c/plugins/commands/config/JukeboxCommands.config.js#L25

xadll commented 7 months ago

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.

https://github.com/lythx/trakman/blob/fc8d4b0bdbdf74464eaeb0f6638bf9c3c171789c/plugins/commands/config/JukeboxCommands.config.js#L25

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.