RSS-Bridge / rss-bridge

The RSS feed for websites missing it
https://rss-bridge.org/bridge01/
The Unlicense
7.37k stars 1.04k forks source link

Splitting YoutubeBridge #3271

Open em92 opened 1 year ago

em92 commented 1 year ago

At the moment, if you visit some youtube profile you will be able to see at least these tabs.

In current implementation, YoutubeBridge is expected to return latest videos from all tabs. At the moment of writing if we are not using duration_min/max params it does like expected [1]. If not, YoutubeBridge returns videos from "Videos" tab [2]. In given link am using very big number, that will include probably any video. Using profile, that posts mostly shorts and has only 2 videos in "Videos" tab.

[1] https://feed.eugenemolotov.ru/?action=display&bridge=YoutubeBridge&context=By+custom+name&custom=%40unreal_keanu&duration_min=&duration_max=&format=Html [2] https://feed.eugenemolotov.ru/?action=display&bridge=YoutubeBridge&context=By+custom+name&custom=%40unreal_keanu&duration_min=&duration_max=99999999&format=Html

As conclusion I would like to do this:

Opinions?

Related issue that is wrongly closed atm. https://github.com/RSS-Bridge/rss-bridge/issues/2308

dvikan commented 1 year ago

I'd prefer to have a single bridge for all the youtube variants. If it makes it easier to code and maintain, sure go ahead and split out into multiple bridges. I don't use the youtube bridge much though. Need opinions from others.

karrakoliko commented 1 year ago

I see it just as a checkboxes in UI for each type of content. I want to paste youtube link, and expect rss bridge to offer me relevant checkboxes: videos, clips, live streams #3216, playlists, whatever it can fetch from given link, and then i can select what exactly i need.

As an and user i dont need separate bridge for each category, it is inconvenient

em92 commented 1 year ago

I see it just as a checkboxes in UI for each type of content.

@vtuu, I already mentioned, that every tab should have different checkboxes.

"YoutubeShortsTab" won't have "duration_min/max" params, since all videos are short. YoutubeBroadcastsTab will also have option to exclude future and current broadcasts

dvikan commented 1 year ago

btw i have refactored and improve the youtube bridge