Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[YoutubeBridge] Playlist bug #647

Closed
em92 opened this issue Mar 14, 2018 · 3 comments · Fixed by #648, #1117 or #1162
Closed

[YoutubeBridge] Playlist bug #647

em92 opened this issue Mar 14, 2018 · 3 comments · Fixed by #648, #1117 or #1162

Comments

@em92
Copy link
Contributor

em92 commented Mar 14, 2018

In playlist items are not always sorted by published date of video. So if playlist has more than 10 items, and latest video is the last, then that latest video won't appear in feed.

Had an PR #644 to remove that limit of 10 for playlists, but it makes script run very long on large playlists, so revoked it myself.
@em92 em92 mentioned this issue Mar 14, 2018
Merged
@em92
Copy link
Contributor Author

em92 commented Mar 19, 2018

Recent PR affects on playlists with not more than 15 videos. I mentioned there:

Partially fixes #647

Example:
Feed: https://feed.eugenemolotov.ru/dev/?action=display&bridge=Youtube&p=PLF2F78607DF9F33FF&format=Html
Original playlist: https://www.youtube.com/playlist?list=PLF2F78607DF9F33FF (23 videos)
Original playlist has https://www.youtube.com/watch?v=EBW_R61WGYU&t=1s&list=PLF2F78607DF9F33FF&index=23 but feed doesn't.

I suggest to reopen this issue.

@em92
Copy link
Contributor Author

em92 commented Mar 19, 2018

Forgot to mention. Last video has upload date - november 8, 2007, videos in feed have dates like from september to november 2007. So mentioned video should be included with other feed items.

@em92 em92 mentioned this issue Nov 12, 2018
Merged
@teromene
Copy link
Member

Sorry, it closed itself automatically when I merged the PR because the text contains "fixes ..."

@teromene teromene reopened this Nov 12, 2018
@sysadminstory sysadminstory mentioned this issue Apr 30, 2019
Merged
teromene pushed a commit that referenced this issue May 8, 2019
@sysadminstory @teromene
[YoutubeBridge] Playlist bug fix (#1117)
b764204 
This commit allow the bridge to parse an infinite number of items of a
Youtube playlist.

It should fix #647 !
@em92 em92 mentioned this issue Jun 10, 2019
Merged
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this issue Apr 17, 2020
@sysadminstory @teromene
[YoutubeBridge] Playlist bug fix (RSS-Bridge#1117)
3819bd5 
This commit allow the bridge to parse an infinite number of items of a
Youtube playlist.

It should fix RSS-Bridge#647 !
@mtfurlan mtfurlan mentioned this issue Nov 25, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@teromene @em92