Only show instances with api enabled when redirecting an api link #31
+12
−7
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Please write a description and give examples please. |
Oups, sorry. I added a description |
unixfox
requested review from
unixfox,
TheFrenchGhosty,
SamantazFox and
syeopite
ChunkyProgrammer
force-pushed
the
only-show-instances-with-api-enabled-for-api-links
branch
from
a0c1217
to
3abfb01
Compare
ChunkyProgrammer
force-pushed
the
only-show-instances-with-api-enabled-for-api-links
branch
from
3abfb01
to
5b82af1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If redirecting an api url then instances that have the api disabled are filtered out.
An example of this being useful is the api documentation, there is a link to redirect an api url here: https://docs.invidious.io/api/#fields (which will redirect to this url: https://redirect.invidious.io/api/v1/videos/aqz-KE-bpKQ?fields=videoId,title,description&pretty=1 )
Before this PR, yewtu.be which has the api disabled would be shown when redirecting api urls. After this PR, yewtu.be is filtered out as well as any other future instances that decide to disable the api.
Test cases:
Redirect an api url:
yewtu.be is not shown
Redirect a video url:
yewtu.be is shown