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

File Download animation continues after failed download #1451

Open
rocodes opened this issue Mar 22, 2022 · 2 comments
Open

File Download animation continues after failed download #1451

rocodes opened this issue Mar 22, 2022 · 2 comments
Labels
good first issue Good for newcomers

Comments

@rocodes
Copy link
Contributor

rocodes commented Mar 22, 2022

Description

When a file download fails, the flashed/banner message appears in the top pane indicating a failure, but the file download animation continues.

Steps to Reproduce

in downloads.py, modify FileDownloadJob.call_api() to throw an immediate error:

    def call_download_api(self, api: API, db_object: File) -> Tuple[str, str]:
        """
        Override DownloadJob.
        """
        raise DownloadException(
                f"Failed to download {db_object.uuid}", type(db_object), db_object.uuid
            )

Expected Behavior

Error banner message is displayed, file download animation stops, and "encrypted file on server" is shown

Actual Behavior

Error banner message is displayed, file download animation continues, and "Downloading..." continues to be displayed.

download_failed

@gonzalo-bulnes gonzalo-bulnes self-assigned this Sep 21, 2022
@rocodes
Copy link
Contributor Author

rocodes commented Nov 1, 2022

@gonzalo-bulnes Is this currently on your plate? Any objections if I label it as a good first issue before the hackathon?

@gonzalo-bulnes gonzalo-bulnes removed their assignment Nov 1, 2022
@gonzalo-bulnes
Copy link
Contributor

Sounds good @rocodes 👍

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

No branches or pull requests

2 participants