Splitting YoutubeBridge

Open
#3,271 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
php
Domain
backend

Research direction

Start by reviewing the existing YoutubeBridge entry point and the related issue #2308. Check how RSS-Bridge handles hiding contexts, then define the shared behavior for the proposed video, playlist, broadcast, and Shorts bridges. Done means each tab has the requested bridge behavior and YoutubeBridge retains its channel-wide feed semantics.

Written by the indexing model from the issue text.

Description

Bridge-Request

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

  • Videos
  • Playlists
  • Broadcasts. Includes current, past broadcasts and future broadcasts
  • Shorts
  • Community. Bridge for this tab already exists which is maintained by @VerifiedJoseph

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:

  • make bridges "YouTubeVideosTab", "YouTubePlaylistsTab", "YouTubeBroadcastsTab" and "YoutubeShortsTab" to output corresponding tabs. All bridges will have common code. "YoutubeShortsTab" won't have "duration_min/max" params, since all videos are short. YoutubeBroadcastsTab will also have option to exclude future and current broadcasts.
  • current "YoutubeBridge" will continue to return latest videos from channel. Feed in [2] should be consider as a bug in YoutubeBridge, and expected behavior in YouTubeVideosTabBridge.
  • "Playlist" option in YoutubeBridge should be hidden. YouTubePlaylistsTab and "Playlist" option in Youtube bridge should have same code. In short both should work. I am not sure if hiding context is available in current implementation of RSS-Bridge.

Opinions?

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

Dominant language
PHP
Stars
9.2k
Forks
1.2k
Avg merge
12h 45m
Merged PRs (30d)
2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from RSS-Bridge/rss-bridge

All issues in RSS-Bridge/rss-bridge

Similar issues

More PHP issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.