-
Notifications
You must be signed in to change notification settings - Fork 757
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 DeArrow support for ft-list-video titles #3688
Add DeArrow support for ft-list-video titles #3688
Conversation
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.
Better naming
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.
Current name does not reflect what it does when reading in code
Co-authored-by: PikachuEXE <pikachuexe@gmail.com>
Co-authored-by: Ajay Ramachandran <dev@ajay.app>
Co-authored-by: PikachuEXE <pikachuexe@gmail.com>
Co-authored-by: PikachuEXE <pikachuexe@gmail.com>
I'm wondering if we should merge this before RC/Release or after like the Piped comments PR |
Imo this can be merged before the RC as it's a small change that adds more functionality to the already existing SponsorBlock API. I guess it depends on when we actually do a RC |
Also it's disabled by default |
Something to note: In the past people complained to us that FT was connecting to Cloudflare telemetry/logging services when it was actually SB doing that. SB is not doing that anymore but DeArrow is also making connections to Cloudflare, idk if its enabled on that side? @ajayyy is the logging enabled or disabled for DeArrow? |
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.
LGTM
Edit: #3688 (comment)?
Maybe change the setting name
to
or
or
|
I like |
fb7689f
* development: (21 commits) Translated using Weblate (Portuguese) Translated using Weblate (Portuguese (Portugal)) Translated using Weblate (Czech) Translated using Weblate (French) Translated using Weblate (Norwegian Bokmål) Translated using Weblate (Japanese) Translated using Weblate (Spanish) Translated using Weblate (Hebrew) Translated using Weblate (Arabic) Translated using Weblate (Italian) Translated using Weblate (Chinese (Traditional)) Translated using Weblate (Chinese (Simplified)) Translated using Weblate (Ukrainian) Translated using Weblate (Hungarian) Translated using Weblate (Turkish) Add DeArrow support for ft-list-video titles (FreeTubeApp#3688) Utils: Don't provide `ytdl://` protocol links to external players (FreeTubeApp#3720) Bump electron from 22.3.14 to 22.3.15 (FreeTubeApp#3723) Bump webpack from 5.88.0 to 5.88.1 (FreeTubeApp#3722) Bump eslint from 8.43.0 to 8.44.0 (FreeTubeApp#3724) ... # Conflicts: # src/renderer/components/ft-list-video/ft-list-video.js
* feature/playlist-2023-05: (21 commits) Translated using Weblate (Portuguese) Translated using Weblate (Portuguese (Portugal)) Translated using Weblate (Czech) Translated using Weblate (French) Translated using Weblate (Norwegian Bokmål) Translated using Weblate (Japanese) Translated using Weblate (Spanish) Translated using Weblate (Hebrew) Translated using Weblate (Arabic) Translated using Weblate (Italian) Translated using Weblate (Chinese (Traditional)) Translated using Weblate (Chinese (Simplified)) Translated using Weblate (Ukrainian) Translated using Weblate (Hungarian) Translated using Weblate (Turkish) Add DeArrow support for ft-list-video titles (FreeTubeApp#3688) Utils: Don't provide `ytdl://` protocol links to external players (FreeTubeApp#3720) Bump electron from 22.3.14 to 22.3.15 (FreeTubeApp#3723) Bump webpack from 5.88.0 to 5.88.1 (FreeTubeApp#3722) Bump eslint from 8.43.0 to 8.44.0 (FreeTubeApp#3724) ...
Add DeArrow support for ft-list-video titles
Pull Request Type
Description
DeArrow is offered through the SponsorBlock api to remove clickbaity titles and thumbnails. This PR adds support for the titles as the thumbnails are more complicated and require communicating with another server.
Future enhancements:
Testing
Desktop