-
Notifications
You must be signed in to change notification settings - Fork 602
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
fix(deps): update dependency youtubei.js to v13.1.0 #3015
Conversation
bro stole renovate's job 💀 |
I was meaning for this to be like a hotfix. When renovate kicks in I intend to close this haha |
Hey just for my own information, which of the fixes in v13.1.0 fixes this downloader issue? |
downloader: Might be LuanRT/YouTube.js@5aecd0a |
Did some more research and apparently it is because of a Jintr bug (didn't know it before today): LuanRT/Jinter#29 |
As a note, the downloader plugin still breaks on a playlist download with > 100 tracks. PR #3022 filters out the additional item type, and resolves the download issues. |
I'm kinda new to GitHub, can you tell me how to apply this fix? |
Uhh, you want the lazy way or the proper way? |
lazy way please? |
@ArjixWasTaken Don't worry, both are fine. |
Fixes #3007
Also fixes #2978