fix: filter members-only videos from feeds #12074
Conversation
YouTube has started to mix members-only videos into the normal videos in the Videos tab. This filters them, as they cannot be watched without an account, leading to clutter in the feed. Requires TeamNewPipe/NewPipeExtractor#1280. Closes: TeamNewPipe#12011 Closes: TeamNewPipe#12040
ShareASmile
added
feature request
|
Is this optional? If not, this shouldn't be merged. The video could still be watched in a browser (possibly on another device). |
|
It’s probably better to display some kind of greyed-out version that cannot be clicked as long as we are not able to extract info for the member-only videos, but still list them in the stream (so they can be opened in the browser). Since the upstream change to the extractor is not in yet, I’d close this for now, but thanks for the contribution! |
|
Why not just adding a checkbox in the "Show/Hide streams" menu? We have already a checkbox there for "Upcoming", so it would make sense to add another one for "Members-only" (enabled by default) |
|
I'm strongly against adding preferences, so if that's the preferred solution, I won't continue to work on it. |
|
I agree with you on not adding preferences, but I think a checkbox in the Show/Hide Streams menu is more easily discoverable |
5 participants
What is it?
Description of the changes in your PR
YouTube has started to mix members-only videos into the normal videos in the Videos tab. This filters them, as they cannot be watched without an account, leading to clutter in the feed.
Before/After Screenshots/Screen Record
Fixes the following issue(s)
Closes: #12011
Closes: #12040
Relies on the following changes
TeamNewPipe/NewPipeExtractor#1280.
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.
Due diligence