-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add loading bar to sapling params download #511
Conversation
✅ Deploy Preview for cheery-moxie-4f1121 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
It doesn't seem to move for me (Firefox, MacOS, Netlify build). The bar started at around 5% (visible in video), however, it stayed there for the entirety of the download, only zipping to 100% instantly after about 2 minutes of waiting, then moving on to the Tx creation. Ideally, we want this to be as granular as possible, like our Shield Sync or minimally Tx Creation (which, although less granular, is still very capable of keeping the user updated and remove the "stuck" feeling). Screen.Recording.2025-01-02.at.3.11.14.PM.mov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK 965aba1
Clean implementation, exceedingly smooth from a UX perspective, this is the last step we needed to finalise our Shield UX.
Abstract
Add loading bar to sapling params download.
This uses the prerelease of pivx shield v1.2.0-1, which has been published using the github artifacts of PIVX-Labs/pivx-shield#97
Testing
Closes #509