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

Correctly handle failed downloads #223

Open
Tracked by #229
dromzeh opened this issue Jun 13, 2023 · 1 comment
Open
Tracked by #229

Correctly handle failed downloads #223

dromzeh opened this issue Jun 13, 2023 · 1 comment
Assignees
Labels
bug Something isn't working priority: high Issue needs to be fixed ASAP

Comments

@dromzeh
Copy link
Member

dromzeh commented Jun 13, 2023

When downloading multiple files, there is a random possibility where some files fail to download.

Failed downloads should be correctly handled by storing the failed downloads - after the download of the remaining files, the failed files should attempt to re-download.

In the case that this still errors, there should be suitable error logging & inform that certain files failed to download (including which ones(?))

There should also be a popup window dependent on users UA to prevent unnecessary downloads in the first place which are bound to error.

@dromzeh dromzeh added bug Something isn't working priority: high Issue needs to be fixed ASAP labels Jun 13, 2023
@dromzeh dromzeh self-assigned this Jun 13, 2023
@dromzeh dromzeh added this to Pipeline Jun 13, 2023
@dromzeh dromzeh moved this to In Progress in Pipeline Jun 13, 2023
@dromzeh dromzeh moved this from In Progress to Stalled in Pipeline Jun 21, 2023
@dromzeh
Copy link
Member Author

dromzeh commented Jul 3, 2023

will probably set UA popup window to display before each download, with the option to not see the message again - handle failed downloads w/ a reusable random component for errors

maybe re-implement sentry with as little tracking as possible, create uuids for rare errors that occur when downloading all/multiple and give users the option to report them that way

@dromzeh dromzeh moved this from Stalled to Backlog in Pipeline Jul 8, 2023
@dromzeh dromzeh moved this from Backlog to Todo in Pipeline Jul 14, 2023
@dromzeh dromzeh moved this from Todo to Stalled in Pipeline Jul 14, 2023
@dromzeh dromzeh moved this from Stalled to Todo in Pipeline Nov 25, 2023
@dromzeh dromzeh moved this from Todo to Stalled in Pipeline Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: high Issue needs to be fixed ASAP
Projects
Status: Stalled
Development

No branches or pull requests

1 participant