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

fix: added a refresh interval if download status is in progress #3275

Merged
merged 3 commits into from Feb 15, 2023

Conversation

OwsleyJr
Copy link
Collaborator

@OwsleyJr OwsleyJr commented Jan 25, 2023

Description

Progress bar would not fetch new download status progress if page was not refocused or reloaded. To solve this, I added a refreshInterval if there is a download to watch. I also added in a change to make the download sync editable, in case you want to grab the download status at a faster rate.

  • Each component we will call refreshIntervalHelper to see if downloadStatus contains items and then update the timer accordingly.
  • Timer set to 5000ms if items are present but this can be altered if needed.
  • Download sync schedule is now editable. Editable times are 30, 45, and 60 seconds.

To-Dos

  • Successful build yarn build
  • Translation keys yarn i18n:extract

Issues Fixed or Closed

@cypress
Copy link

cypress bot commented Jan 25, 2023

Passing run #1513 ↗︎

0 28 0 0 Flakiness 0

Details:

fix: added a refresh interval if download status is in progress
Project: overseerr Commit: a066b7a353
Status: Passed Duration: 03:33 💡
Started: Feb 15, 2023 2:17 PM Ended: Feb 15, 2023 2:21 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@OwsleyJr OwsleyJr force-pushed the fix/progress-bar-updating branch 7 times, most recently from 7cff47f to 895c008 Compare January 26, 2023 18:16
server/lib/settings.ts Outdated Show resolved Hide resolved
src/components/MovieDetails/index.tsx Outdated Show resolved Hide resolved
@OwsleyJr OwsleyJr force-pushed the fix/progress-bar-updating branch 3 times, most recently from 14ea136 to eb4c8ee Compare January 27, 2023 01:45
@OwsleyJr OwsleyJr force-pushed the fix/progress-bar-updating branch 2 times, most recently from 684a8fb to 2f1d500 Compare February 12, 2023 02:43
server/job/schedule.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@danshilm danshilm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 🎉

@danshilm danshilm merged commit 1e2c6f4 into develop Feb 15, 2023
@danshilm danshilm deleted the fix/progress-bar-updating branch February 15, 2023 15:16
@github-actions
Copy link

🎉 This PR is included in version 1.33.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

lenaxia pushed a commit to lenaxia/overseerr-oidc that referenced this pull request Jan 3, 2024
…3275)

* fix: added a refresh interval if download status is in progress

* refactor: switched to a function instead of useEffect

* feat: added editable download sync schedule
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Status badge shows inaccurate progress when updating without page refresh
2 participants