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

Restoring new wallet fails #11647

Closed
NejcZdovc opened this issue Sep 10, 2020 · 5 comments · Fixed by brave/brave-core#6610
Closed

Restoring new wallet fails #11647

NejcZdovc opened this issue Sep 10, 2020 · 5 comments · Fixed by brave/brave-core#6610

Comments

@NejcZdovc
Copy link
Contributor

Description

We need to handle this error as success and return 0 balance.

[ RESPONSE - OnRequest ]
> Url: https://grant.rewards.brave.software/v3/wallet/uphold/b60b2138-839a-4203-9fb5-b8d34c0ea531
> Result: Failure
> HTTP Code: 400
> Body: {"message":"wallet not capable of balance inquiry","code":400}

Steps to Reproduce

  1. Clean profile
  2. Enable rewards
  3. Open backup dialog
  4. Get recovery keys from local storage
  5. Restore wallet

Actual result:

Restore is successful, but you get server error

Expected result:

No server error

Reproduces how often:

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@LaurenWags
Copy link
Member

LaurenWags commented Sep 25, 2020

@NejcZdovc could you clarify what you mean by

Get recovery keys from local storage

in the STR?

@NejcZdovc
Copy link
Contributor Author

@LaurenWags sadly this use case is not viable anymore as we removed it from local storage as well. So you will just need recovery keys from older wallet

@LaurenWags
Copy link
Member

@NejcZdovc will any 24 word wallet work as long as it has a zero balance?

@NejcZdovc
Copy link
Contributor Author

@LaurenWags yes it will

@LaurenWags
Copy link
Member

LaurenWags commented Sep 30, 2020

Verified passed with

Brave | 1.16.47 Chromium: 86.0.4240.55 (Official Build) dev (x86_64)
-- | --
Revision | a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS | macOS Version 10.14.6 (Build 18G3020)
[ RESPONSE - OnRequest ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/............................
> Result: Success
> HTTP Code: 200
> Body: {"total":0,"spendable":0,"confirmed":0,"unconfirmed":0}

Verification passed on

Brave | 1.16.47 Chromium: 86.0.4240.55 (Official Build) dev (64-bit)
-- | --
Revision | a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

  • Verified STR from description using a wallet with BAT balance
[ RESPONSE - OnRequest ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/..........
> Result: Success
> HTTP Code: 200
> Body: {"total":16,"spendable":16,"confirmed":16,"unconfirmed":0}

  • Verified STR from description using a wallet with 0 BAT balance
[ RESPONSE - OnRequest ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/..................
> Result: Success
> HTTP Code: 200
> Body: {"total":0,"spendable":0,"confirmed":0,"unconfirmed":0}

Verification passed on

Brave 1.16.50 Chromium: 86.0.4240.55 (Official Build) dev (x86_64)
Revision a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS Ubuntu 18.04 LTS
[ RESPONSE - OnRequest ]
> Url: https://grant.rewards.bravesoftware.com/v3/wallet/uphold/
> Result: Success
> HTTP Code: 200
> Body: {"total":0,"spendable":0,"confirmed":0,"unconfirmed":0}

@rebron rebron changed the title [Desktop] Restoring new wallet fails Restoring new wallet fails Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment