search

jack3898 / discord-youtube-bot-2

A powerful Discord YouTube Bot.
0 stars 0 forks source link

Added 13 digit id clause to playlist regexp #1

Closed brianetaveras closed 2 years ago

brianetaveras commented 2 years ago

Did not see any template for contributions, so I'm just going to open a pr. Let me know if changes are needed!

Problem

Youtube mix playlists are not supported due to the ID validation regexp validating 34 characters.

Solution

Confirmed that youtube mix playlists IDs have a length of 13 and adjusted the regexp to evaluate for 34 or 13.

Hope this is helpful :)

jack3898 commented 2 years ago

Hey @brianetaveras thanks for that, yes it is helpful! I had no idea. PR looks nice a simple as well so I will just go an merge it in.

Also yeah, no contribution template. I think I should really do that as some point!

Again, thanks. 😊