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

Add loading bar to sapling params download #511

Merged
merged 7 commits into from
Jan 7, 2025

Conversation

Duddino
Copy link
Member

@Duddino Duddino commented Jan 2, 2025

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

  • Make a shield tx
  • Assert that the shield params loading bar moves

Closes #509

Copy link

netlify bot commented Jan 2, 2025

Deploy Preview for cheery-moxie-4f1121 ready!

Name Link
🔨 Latest commit 965aba1
🔍 Latest deploy log https://app.netlify.com/sites/cheery-moxie-4f1121/deploys/677cf427c0d8c800089a29ca
😎 Deploy Preview https://deploy-preview-511--cheery-moxie-4f1121.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JSKitty JSKitty added the Enhancement New feature or request label Jan 2, 2025
@JSKitty
Copy link
Member

JSKitty commented Jan 2, 2025

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

Copy link
Member

@JSKitty JSKitty left a 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.

@Duddino Duddino merged commit f25c800 into PIVX-Labs:master Jan 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shield Params should use our stream-download system
2 participants