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

Warn users that the recovery key / backup does not actually back up their BAT balance (only their estimated ad earnings) #10021

Closed
Miyayes opened this issue May 31, 2020 · 5 comments · Fixed by brave/brave-core#5958

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented May 31, 2020

Description

Since v1.3, BAT is stored and calculated locally as part of the browser's data. However, the backup/recovery key option still exists in the browser and leads users to believe that it will back up their BAT balance. In reality, the recovery key only backs up their "estimated ad earnings this month" counter, which is determined server-side.

This is deeply confusing to users, and many users are left puzzled and upset after they find out that they can only restore 0.0 BAT of their previous balance.

A very large number of users have been affected by this. For one example: https://www.reddit.com/r/BATProject/comments/gu182q/all_my_bats_are_gone_after_recovery/

Possible Solution

Although this may be solved in the future with a later implementation of Sync v2 that works with Brave Rewards data, an interim solution is required.

For example, we can provide appropriate messaging warning users of the recovery feature and its limits.

A more limiting solution would be to hide or remove the backup feature completely (like on iOS and Android).

@Miyayes
Copy link
Collaborator Author

Miyayes commented Jun 1, 2020

@cryptojennie
Copy link

@Brave-Matt
Copy link

@LaurenWags
Copy link
Member

Marking as QA/Blocked until able to sync up with @mandar-brave

@LaurenWags
Copy link
Member

LaurenWags commented Aug 6, 2020

Verified passed with

Brave	1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision	a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS	macOS Version 10.14.6 (Build 18G3020)

Verified test plan from brave/brave-core#5958
For Test plan 1 from the spec:

Screen Shot 2020-08-06 at 9 56 38 AM

For Test plan 2 from the spec:

  • confirmed text on modal matches spec
  • confirmed clicking the Uphold link opens login page to Uphold

Screen Shot 2020-08-06 at 9 54 12 AM

The following are existing open/related issues (confirmed with @NejcZdovc & @mandar-brave as appropriate)

Also spot checked the following upgrade scenarios:

  1. Upgrade with only VG, opened got correct backup modal (no words displayed)
  2. Upgrade with VG + user funds, got correct backup modal (words displayed)
  3. Upgrade with only user funds, got correct backup modal (words displayed)
  4. Upgrade with only Uphold, got correct backup modal (no words displayed)

Verification passed on


Brave | 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
-- | --
Revision | a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS | Windows 10 OS Version 1903 (Build 18362.959)


For Test plan 2 from the spec:

  • confirmed text on modal matches spec
  • confirmed clicking the Uphold link opens login page to Uphold
    image

Verification passed on

Brave 1.12.107 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS Ubuntu 18.04 LTS

For Test plan 2 from the spec:

  • confirmed text on modal matches spec
  • confirmed clicking the Uphold link opens login page to Uphold
    image

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