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

Display error in WebUI when download fails #21696

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Piccirello
Copy link
Member

@Piccirello Piccirello commented Oct 27, 2024

Previously we would still download the file but it would contain the error response, resulting in an invalid file.

To test: export a .torrent file for a torrent that hasn't yet downloaded metadata

@Piccirello Piccirello added the WebUI WebUI-related issues/changes label Oct 27, 2024
@Piccirello Piccirello added this to the 5.1 milestone Oct 27, 2024
@Piccirello Piccirello requested a review from a team October 27, 2024 20:39
@Piccirello Piccirello force-pushed the webui-torrent-export branch 2 times, most recently from 7af8ea9 to 91084b6 Compare November 3, 2024 21:44
@Piccirello Piccirello requested review from Chocobo1 and a team November 3, 2024 21:44
src/webui/www/private/scripts/misc.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/misc.js Outdated Show resolved Hide resolved
Copy link
Member

@Chocobo1 Chocobo1 left a comment

Choose a reason for hiding this comment

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

Should be the last round of review.

src/webui/www/private/scripts/misc.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/misc.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/misc.js Outdated Show resolved Hide resolved
Previously we would still download the file but it would contain the error response.

Signed-off-by: Thomas Piccirello <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebUI WebUI-related issues/changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants