Skip to content
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

YouTube Support #74

Open
sidit77 opened this issue Jul 25, 2023 · 3 comments
Open

YouTube Support #74

sidit77 opened this issue Jul 25, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@sidit77
Copy link
Contributor

sidit77 commented Jul 25, 2023

I was recently thinking about adding YouTube support. I have a working prototype here using Piped including support for clips and timestamped links,

However, I think Google has a problem with 3rd party YouTube clients in the PlayStore. So this feature would have to be limited to F-Droid and Github. Do you think this could be done using some kind of conditional compilation or should I just keep this as a seperate branch and only upstream the generic changes I had to make to implement YouTube support?

@SpartanJ
Copy link
Owner

Oh that's awesome!
This can be achieved with android build variants / flavors. That would be the correct way of doing it. I'm exited that you took the work on it. I thought of doing it several times but I'm too lazy and I'm currently paying for the premium account so I kinda lost interest. Hope you can continue working on it and we can merge it to this repo. Good luck!

@SpartanJ SpartanJ added the enhancement New feature or request label Jul 26, 2023
@sidit77
Copy link
Contributor Author

sidit77 commented Jul 26, 2023

Thanks for the quick reply. I have another question: How do you generally handle video quality? Just selecting the highest seems risky given that YouTube supports 4k videos. Maybe there should be a setting to set an upper limit like 1080p?

@SpartanJ
Copy link
Owner

Currently I select the highest resolution up to 1080p, ignoring anything bigger than that. But ideally I should add an option to prioritise a user-configured resolution, but I was too lazy to implement it. You can keep the same logic for the moment, use the greater resolution up to 1080p.

@sidit77 sidit77 mentioned this issue Aug 1, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants