Skip to content

Conversation

@franz-dc
Copy link
Contributor

@franz-dc franz-dc commented Feb 22, 2025

Fixes #3007
Also fixes #2978

@ArjixWasTaken
Copy link
Member

bro stole renovate's job 💀

@franz-dc
Copy link
Contributor Author

I was meaning for this to be like a hotfix. When renovate kicks in I intend to close this haha

@JasperSurmont
Copy link

Hey just for my own information, which of the fixes in v13.1.0 fixes this downloader issue?

@ArjixWasTaken
Copy link
Member

ArjixWasTaken commented Feb 25, 2025

downloader: Might be LuanRT/YouTube.js@5aecd0a
playlist: LuanRT/YouTube.js@a3fafe2

@JasperSurmont
Copy link

Did some more research and apparently it is because of a Jintr bug (didn't know it before today): LuanRT/Jinter#29

@DeathCamel58
Copy link
Contributor

DeathCamel58 commented Feb 25, 2025

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.

@ThatSidus
Copy link

I'm kinda new to GitHub, can you tell me how to apply this fix?

@ArjixWasTaken
Copy link
Member

@ThatSidus

Uhh, you want the lazy way or the proper way?

@akaylaxman
Copy link

lazy way please?

@ThatSidus
Copy link

@ArjixWasTaken Don't worry, both are fine.

@JellyBrick JellyBrick added the dependencies Pull requests that update a dependency file label Feb 28, 2025
@JellyBrick JellyBrick merged commit dbea4c5 into pear-devs:master Feb 28, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Downloader wont download [Bug]: Plugin Downloader doesnt fetch entire playlist, but exactly 100 tracks, if Playlist has more than 100 tracks

7 participants