miniflux / v2

Minimalist and opinionated feed reader
https://miniflux.app
Apache License 2.0
6.45k stars 702 forks source link

Feed detection from a Youtube Playlist should also cover playlists URLs that contain a video ID #2628

Closed SansGuidon closed 2 weeks ago

SansGuidon commented 2 months ago

The recent work done for detecting feeds from Youtube playlists is great and I enjoy it. However I had an error the first time I tested it because I had different expectations which motivates me to write this feature request.

Sometimes while opening a playlist, we start immediately with a playing video and thus the URL will be focused on the video while containing also the Playlist ID at the end, e.g :

https://www.youtube.com/watch?v=xF81tWZX84c&list=PLWrIqGszZr5kGyHwMPQBfKnRd850ynGBh

And then there is the pattern supported by Miniflux, which is the canonical playlist URL, like https://www.youtube.com/playlist?list=PLWrIqGszZr5kGyHwMPQBfKnRd850ynGBh

Notice that the Playlist ID is present in both examples, but only the second example is supported by Miniflux currently.

To reproduce the UX : when Youtube shows playlists as part of research results, clicking on one DOM element instead of another will either redirect to the playlist home page (color purple in image below) or will open the URL to a specific video and append the playlist ID to the query parameters (the URL in red in the picture below).

image

SansGuidon commented 2 weeks ago

Awesome ! Many thanks @ztec 🤟🏻