-
-
Notifications
You must be signed in to change notification settings - Fork 207
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.
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
Video frontend Revamp #274
Video frontend Revamp #274
Conversation
I do agree with re-working the page and recommending things like Newpipe/Freetube. |
Oh dear, I need to fix the PR |
✔️ Deploy Preview for privacyguides ready! 🔨 Explore the source changes: 253bf5e 🔍 Inspect the deploy log: https://app.netlify.com/sites/privacyguides/deploys/61960108ec486b0008d2b721 😎 Browse the preview: https://deploy-preview-274--privacyguides.netlify.app/video-streaming |
a71ef5f
to
31ca570
Compare
This PR is ready for merge. It also fixes #334 PeerTube is not mentioned as I cannot find how you can have a private subscription list with it. It just looks like Youtube, but federated. You still need to sign up for an account with an instance and what not. LBRY is a much better alternative as it stores the subscription list on your local LBRY wallet. |
Co-authored-by: Daniel Nathan Gray <dng@disroot.org>
Co-authored-by: Daniel Nathan Gray <dng@disroot.org>
Co-authored-by: Daniel Nathan Gray <dng@disroot.org>
This addition should've been on this same PR before it was merged into main. Pushing incomplete contributions into main, defeats the purpose of stable-branches. Some people were confused by this, so please try to avoid it in the future |
Regarding this post specifically. I'll post my reply. |
The video page is extremely wrong and broken right now:
This page is so much of a mess that it needs to be completely redone, and I don't think keeping it is a good idea considering it's a legacy page. We will make a new page recommending things like Newpipe/Freetube later using the new system.