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

Shield Params should use our stream-download system #509

Open
JSKitty opened this issue Dec 16, 2024 · 0 comments
Open

Shield Params should use our stream-download system #509

JSKitty opened this issue Dec 16, 2024 · 0 comments
Labels
Enhancement New feature or request

Comments

@JSKitty
Copy link
Member

JSKitty commented Dec 16, 2024

Currently, Shield Params are the biggest 'UX bottleneck' for users - with users frequently citing confusion due to the long, seemingly "frozen" state of "Loading SHIELD Params..." and a progress bar that doesn't move.

Even if the params themselves are useless unless downloaded in full (thus, there's no functional benefit of streaming them), the benefit would purely be for UX purposes, letting us display the download in MBs as well as display a real-time progress bar.

Ideally, although this could be a separate PR: we should also try saving Shield Params within IndexedDB, some users report issues of the Cache seemingly not working, and I assume it's due to either using a locked down browser (stricter cache policies) or simply their device is clearing cache frequently - the UX benefit of saving the params permanently is much greater than the small downside of more persistant, but reliable storage usage.

@JSKitty JSKitty added the Enhancement New feature or request label Dec 16, 2024
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

No branches or pull requests

1 participant