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: Par2 repairing status #424

Merged
merged 2 commits into from
Nov 15, 2024
Merged

fix: Par2 repairing status #424

merged 2 commits into from
Nov 15, 2024

Conversation

dnzbk
Copy link
Collaborator

@dnzbk dnzbk commented Nov 15, 2024

Description

issue

  • fixed Par2 repair status showing in the webui

Testing

  • Windows 11

@dnzbk dnzbk requested review from phnzb and luckedea November 15, 2024 04:46
@dnzbk dnzbk merged commit c12ea6d into develop Nov 15, 2024
8 checks passed
@dnzbk dnzbk deleted the fix/par2-repairing-status branch November 15, 2024 12:36
@isarrider
Copy link

Does not fix it for me - still just a dark bar while repairing...

@dnzbk
Copy link
Collaborator Author

dnzbk commented Nov 15, 2024

image
image

It works fine for me. Could you share your STATUS page and nzb file?
denis@nzbget.com

@isarrider
Copy link

hmm interesting...
I tried with a couple of nzbs and it doenst make a difference...
it is fast as hell but doesnt throw infos...
Give me some time to think about it...

beertje44 pushed a commit to beertje44/nzbget that referenced this pull request Dec 4, 2024
@isarrider
Copy link

Actually now it works - weird...
But I don't get a time remaining info when it downloads the par files needed to repair...

@dnzbk
Copy link
Collaborator Author

dnzbk commented Dec 20, 2024

Yes, the information about the remaining Par2 file download time is incorrect but this bug is not caused by the recent changes and has existed since at least v21.1. I’ll look into it when I have some free time.

@dnzbk
Copy link
Collaborator Author

dnzbk commented Dec 23, 2024

@isarrider
Transformed to the issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants