MusicBot icon indicating copy to clipboard operation
MusicBot copied to clipboard

[Feature Request] Better playlist selection

Open the-snesler opened this issue 4 years ago • 5 comments

Is your feature request related to a problem? Please describe.

The playlist feature is rather difficult to activate sometimes, especially for users that aren't incredibly technically inclined. Loading a playlist is a separate command, the playlist title is case sensitive, and it's difficult to differentiate between different playlists in the list command. Invoking !play without the playlist subcommand Invoking !play playlist without proper capitalization The result of !playlist list with many playlists Proper usage of the !play playlist command

What is your ideal solution to the problem?

I feel that there should be a config option to have the bot check the playlist folder before searching YouTube and pull from there instead, if there's a match.

Playlist selection shouldn't be case sensitive. The titles look better capitalized in the !playlist list command, but it's confusing to invoke them with the exact same capitalization, especially if the bot owner or admins (example being myself) are inconsistent with capitalization.

I'm unfortunately not much of a developer, but as far as I understand there's already a paginator used elsewhere that could clean up !playlist list?

How would this feature be used?

The playlist feature, while powerful, likely sees less usage because of these barriers to entry. The attached screenshots are only a couple of examples I've seen in a server with just 400 members.

Making the playlists (optionally) easier to access seems like a benefit for usability & customizability that is lacking in other music bots.

Additional Info

Thank you for this wonderful bot!! Besides this (relatively minor) issue, our community has been loving it!

Checklist

the-snesler avatar Apr 08 '22 23:04 the-snesler

i would say its just easier for them to add in the commands tutorial section !play playlist NAME (case sensitive) and then that whole problem is sorted

purnell569 avatar Apr 09 '22 07:04 purnell569

very true! this is admittedly a lot of work, lol. I do think the playlist list could be improved at any rate

the-snesler avatar Apr 09 '22 16:04 the-snesler

There are 2 solutions for this:

  1. Just like @TetraTsunami suggested, we make the play command look for playlists first, before falling back to searching elsewhere.
  2. We add a playplaylist command. You can then create an alias for it, such as !pp.

Making playlists case-insensitive should be fairly easy to implement as well

MichailiK avatar Apr 09 '22 16:04 MichailiK

We might actually need to go with the second option, once we start migrating JMusicBot to slash commands.

MichailiK avatar Apr 09 '22 17:04 MichailiK

1. Just like @TetraTsunami suggested, we make the `play` command look for playlists first, before falling back to searching elsewhere.

2. We add a `playplaylist` command. You can then create an alias for it, such as `!pp`.

I think we can actually do both of these; using the separate command will exclusively search for playlists (and with slash commands, can even utilize autocomplete). When using normal play, a cached playlist list is checked first. This would also solve the request of people asking for shortcut names for tracks; they could just create playlists with a single track in it if they want to play a specific song for a specific request.

jagrosh avatar May 20 '22 18:05 jagrosh