Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

loading/progress indicator for wallet recovery #11790

Closed
LaurenWags opened this issue Nov 4, 2017 · 8 comments
Closed

loading/progress indicator for wallet recovery #11790

LaurenWags opened this issue Nov 4, 2017 · 8 comments

Comments

@LaurenWags
Copy link
Member

Description

Several users have mentioned that after entering or importing their BAT wallet recovery keys, it doesn't appear as though anything is happening on the UI. They have requested some kind of loading or progress indicator to let the user know something is happening. If a progress indicator isn't possible, maybe some kind of transition overlay (spinning circle?) to let the user know something is going on in the background.

Steps to Reproduce

  1. Have backup words for a BAT wallet.
  2. Create a new profile and enable payments.
  3. Recover your wallet.

Actual result:
Sometimes wallet recovery takes up to 10 minutes, during this time there is no visual indication in the UI that anything is happening.

Expected result:
Some kind of visual notifier that a wallet recovery is being attempted.

Reproduces how often:
unknown. sometimes wallets recover fast, so this isn't an issue. other times it takes up to 10 minutes.

Brave Version

BAT Mercury enabled versions (0.19.48, 0.19.70, etc)

Reproducible on current live release:
yes

Additional Information

@bsclifton
Copy link
Member

Is this server related (ex: see #11590)?

@bsclifton bsclifton added the priority/P3 Major loss of function. label Nov 7, 2017
@bsclifton
Copy link
Member

bsclifton commented Nov 7, 2017

Would adding a "Please wait a few minutes" be fine? Note: while it's loading, we also show 0 USD... maybe it should say "loading" or "please wait" instead of dollar amount

cc: @alexwykoff @bradleyrichter

@alexwykoff
Copy link
Contributor

We're currently investigating a similar problem on the publisher side. $0.00 is distressing so it would make sense to obfuscate the amount until we know for sure it is $0.00 😸

@srirambv
Copy link
Collaborator

srirambv commented May 3, 2018

Blocked on #14014

@kjozwiak
Copy link
Member

kjozwiak commented May 8, 2018

Unfortunately this still doesn't resolve the issue 100%. Sometimes you'll still notice the $0.00 value appear before the real amount is displayed via the modal. Sometimes the transition is instant and sometimes the transition takes a few seconds. I'm guessing this depends on the latency of your network and the ledger server.

@kjozwiak kjozwiak reopened this May 8, 2018
@kjozwiak kjozwiak modified the milestones: 0.22.x Release 3 (Beta channel), 0.23.x (Developer Channel) May 8, 2018
@kjozwiak
Copy link
Member

kjozwiak commented May 8, 2018

Moving this out as it still needs some more improvements. As mentioned above, we're still displaying $0.00 at times when recovering. However, this has improved the experience by letting the user know that we're actually doing a recovery rather than just displaying $0.00 for a longer period of time.

@srirambv
Copy link
Collaborator

srirambv commented Jun 19, 2018

Verified on Windows 10 x64 using

  • 0.23.11 - 6565c06
  • Muon - 7.1.0
  • libchromiumcontent - 67.0.3396.87

Verified with macOS 10.12.6 using

  • 0.23.11 6565c06
  • Muon 7.1.0
  • libchromiumcontent 67.0.3396.87
  • also verified by @kjozwiak

Verified on Ubuntu 17.10 x64

  • 0.23.12 - 88f6f07
  • Muon - 7.1.1
  • libchromiumcontent - 67.0.3396.87

@kjozwiak
Copy link
Member

Great work BTW, this looks and behaves so much better. Huge improvement IMO 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.