-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Add Invidious support #8220
Add Invidious support #8220
Conversation
281562c
to
977bf71
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The icon you are now using for the YouTube service is close to the one which is used by YouTube on their platform.
By seeing that Vanced has been takedown because of their logo very close to the one of YouTube (this is what the Vanced team said), we should not take the risk at all to use a similar logo to the YouTube one, otherwise this will open a door for Google to DMCA the app.
Please use only again the current placeholder.
Please link to https://instances.invidious.io/ instead, since this will always link to the instances list. PS: good job |
This comment was marked as resolved.
This comment was marked as resolved.
@litetex This looks very promising! Here are my two cents about what I think should be changed a little bit: Anyways, thanks for your contribution! |
We don't recommend using anything other than instances.invidious.io since there's plan to rework api/redirect... |
Okay thanks for the hint. I'm a little bit confused since https://api.invidious.io/ is literally linked on https://instances.invidious.io. However, are there other api endpoints to fetch a list of available instances? |
Sorry, I badly explained (and forgot to reply sooner): what I mean is that we don't recommend using anything other than instances.invidious.io as a web interface that users interact with, the json endpoint will always remain (it's made to be stable), but we have plan to make a single web interface (on instances.invidious.io ) to replace both the current markdown file (located in the documentation) and the web interface of the instance api (and ideally redirect.invidious.io), this is why we recommend using instances.invidious.io (and not link to the markdown file directly, or another web interface). |
4275c90
to
f5d4070
Compare
I can't seem to add |
Usually only the invidious instances with |
@FireMasterK Is it possible to add the API? |
No, they've been blocked due to bots abusing them. If there's a way to whitelist NewPipe, I may consider it however. Currently for the time being, only piped allows all traffic. |
was there for testing purposes only
Reloads the current instance from the persistence layer (preferences)
Kudos, SonarCloud Quality Gate passed! |
BTW, what about fetching the actual list of official instances from the Invidious URL and letting the user tap them to add them to Newpipe? Much more user-friendly than manually entering a URL. This way, the user will also avoid mistakenly adding an instance that doesn't provide the API as needed. Edit: Should the Youtube entry be removable? I feel like that might lead to accidental removals by novice users, who won't understand how to get it back without creating undue noise in our repo. The button to restore defaults is right there, but you know how most users are. Edit 2: Is everything fetched through Invidious? As in, do we avoid pinging Youtube at all? It seems like that, because I just refreshed my feed and it was much slower. (Feed is definitely loaded from Invidious, but I don't know about pinging YT at all.)
Edit 4: I forgot to mention: the thumbnail when using Invidious is smaller. It doesn't reach the device edges. There are white/black (based on theme) bars on either side of it. Switching to YT loads the same thumbnail properly. Edit 5: The Description tab text is not formatted when the video details are loaded from Invidious. It's a giant continuous paragraph. Also, live videos aren't marked as such in an Invidious-loaded feed. |
not quite sure where else to put this so ill ask here: edit: |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as duplicate.
This comment was marked as duplicate.
Although this was a really good PR, I'm closing it as the author is not going to contribute to NewPipe anymore. Thanks to everyone involved, we'll try to implement this feature in the future rewritten NewPipe! |
What is it?
Description of the changes in your PR
👉 TeamNewPipe/NewPipeExtractor#833
Adds support for Invidious instances (require a working API).
Configuration demo:
NPYTLikeConfig1Demo.gif.mp4
Fixes the following issue(s)
Relies on the following changes
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.
Due diligence
Todo
/api/v1/videos/:id
- Returnpublished
forrecommendedVideos
iv-org/invidious#3058