Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Export - Add loading state when click on download button#2325

Merged
katspaugh merged 3 commits into
address-book-v2from
feature/export-loading-state
May 27, 2021
Merged

Export - Add loading state when click on download button#2325
katspaugh merged 3 commits into
address-book-v2from
feature/export-loading-state

Conversation

@matextrem

Copy link
Copy Markdown
Contributor

What it solves

There is no feedback for the user when click on download button in order to export the address book.This PR closes #2305

How this PR fixes it

This PR adds a loading spinner when you click on the download button in order to show a feedback to the user that the file is being downloaded.

How to test it

  • Open/Load a safe
  • Go to Address book
  • Click Export button
  • Click on download button

@matextrem matextrem added the Enhancement ✨ Minor Improvement / changes to existing functionality label May 24, 2021
@matextrem matextrem requested review from dasanra and katspaugh May 24, 2021 23:39
@matextrem matextrem self-assigned this May 24, 2021
@matextrem matextrem linked an issue May 24, 2021 that may be closed by this pull request
@github-actions

Copy link
Copy Markdown

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions

github-actions Bot commented May 24, 2021

Copy link
Copy Markdown

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@github-actions

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

@katspaugh katspaugh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works perfectly, thank you!

Could you please revert the yarn.lock changes?

@katspaugh

Copy link
Copy Markdown
Contributor

Noticed a bug: after you've downloaded the CSV once, it won't let you download it again unless you refresh the page.

@matextrem matextrem requested a review from katspaugh May 27, 2021 14:17
@matextrem

Copy link
Copy Markdown
Contributor Author

@katspaugh can you test again ? It's working for me.

@katspaugh katspaugh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works perfectly now 🙏

@katspaugh katspaugh merged commit 0d915a6 into address-book-v2 May 27, 2021
@katspaugh katspaugh deleted the feature/export-loading-state branch May 27, 2021 15:26
@github-actions github-actions Bot locked and limited conversation to collaborators May 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Enhancement ✨ Minor Improvement / changes to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Address Book v2] Add a loading state to the CSV download button

2 participants