Skip to content
This repository was archived by the owner on Jan 21, 2024. It is now read-only.

sponsorblock: move fetching VideoSegment's to DownloadDialog#309

Merged
polymorphicshade merged 1 commit into
polymorphicshade:sponsorblockfrom
bravepipeproject:move-fetching-VideoSegments-to-DownloadDialog
Apr 8, 2023
Merged

sponsorblock: move fetching VideoSegment's to DownloadDialog#309
polymorphicshade merged 1 commit into
polymorphicshade:sponsorblockfrom
bravepipeproject:move-fetching-VideoSegments-to-DownloadDialog

Conversation

@evermind-zz

@evermind-zz evermind-zz commented Feb 17, 2023

Copy link
Copy Markdown

Pushing download button for a video sometimes takes for ages.
This PR moves Sponsorblock VideoSegment logic to DownloadDialog.

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • move fetching VideoSegment's to DownloadDialog
  • integrate ProgressBar to show that we are fetching VideoSegment's

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

- move fetching VideoSegment's to DownloadDialog
- integrate ProgressBar to show that we are fetching VideoSegment's
@polymorphicshade
polymorphicshade merged commit fa86c26 into polymorphicshade:sponsorblock Apr 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants