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

0.00 BAT showing in Rewards panel, even though user is logged in with custodian, and rewards-internals shows Loading... #29627

Closed
Miyayes opened this issue Apr 11, 2023 · 2 comments · Fixed by brave/brave-core#18105
Assignees
Labels

Comments

@Miyayes
Copy link
Contributor

Miyayes commented Apr 11, 2023

Description

Based on investigation with a user on Uphold, this was due to brave.rewards.external_wallet_type missing from Preferences.

Now received a report of the same on bitFlyer.

cc: @A-t-s-u-s-h-i

@Miyayes Miyayes added bug feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. OS/Desktop labels Apr 11, 2023
@Miyayes Miyayes added this to New Issues in Brave Rewards via automation Apr 11, 2023
@Miyayes Miyayes added the needs-investigation A bug not 100% confirmed/fixed label Apr 11, 2023
@Miyayes Miyayes moved this from New Issues to On deck in Brave Rewards Apr 17, 2023
@szilardszaloki szilardszaloki moved this from On deck to In progress in Brave Rewards Apr 17, 2023
@Miyayes Miyayes added priority/P1 A very extremely bad problem. We might push a hotfix for it. and removed priority/P2 A bad problem. We might uplift this to the next planned release. needs-investigation A bug not 100% confirmed/fixed labels Apr 17, 2023
Brave Rewards automation moved this from In progress to Done Apr 18, 2023
@brave-builds brave-builds added this to the 1.52.x - Nightly milestone Apr 18, 2023
@kjozwiak
Copy link
Member

The above requires 1.50.121 or higher for 1.50.x verification 👍

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 19, 2023

Verification PASSED on

Brave | 1.50.121 Chromium: 112.0.5615.138 (Official Build) (64-bit)
-- | --
Revision | 2258f2fcaeb07bc95c992b7ca47f0b43ead9865c-refs/branch-heads/5615@{#1282}
OS | Windows 10 Version 21H2 (Build 19044.2846)

Per brave/brave-core#18105 (comment), verified the test plan from brave/brave-core#17480 (comment)

Uphold - PASSED
1. Before enabling rewards - PASSED
  • Confirmed that there is no Balance info displayed on the brave://rewards-internals page before enabling the rewards

image

2. After enabling rewards - PASSED
  • Confirmed that after enabling the rewards the vBAT balance is shown on the Rewards Internals page
  • Confirmed that the Connect account is shown on the brave://rewards settings page, BR panel and rewards NTP widget
Example Example Example Example
image image image image
3. After connecting a custodial wallet - PASSED
  • Confirmed that after connecting custodian wallet, both the vBAT/external wallet balance is displayed on the Rewards Internals page
  • Confirmed that after connecting custodian wallet, total balance is displayed on the Rewards page, Rewards panel and Rewards NTP widget
Example Example Example Example
image image image image
4. Logged out state - PASSED
  • Confirmed that in the logged out state, the vBAT balance (zero) is shown only on the Rewards Internals page
  • Confirmed that Logged out UI is shown on the brave://rewards, BR panel, Rewards NTP and Tipping banner
  • Confirmed that : ( You are currently logged out text message is shown on the tipping banner
    • Note, tipping banner on Nightly 1.52.x will differ from 1.50.x and 1.51.x, this is because new design for tipping banner was merged to Nightly
Example Example Example Example Example
image image image image image
5. Fail the fetch balance request - PASSED
  • Uphold wallet (pre-req: have Charles Proxy set up to return 500 error for the endpoint (https://api-sandbox.uphold.com/v0/me/cards/*))
  • Confirmed that spinner animation is shown for both the Rewards BAT (vBAT) and external wallet balance on brave://rewards-internals page
Example Example Example Example
image image image image
6. Reset rewards - PASSED
  • Confirmed that after Reset rewards the wallet balance info is not displayed on the brave://rewards-internals page

image

Gemini - PASSED
1. Before enabling rewards - PASSED
  • Confirmed that there is no Balance info displayed on the brave://rewards-internals page before enabling the rewards

image

2. After enabling rewards - PASSED
  • Confirmed that after enabling the rewards the vBAT balance is shown on the Rewards Internals page
  • Confirmed that the Connect account is shown on the brave://rewards settings page, BR panel and rewards NTP widget
Example Example Example Example
image image image image
3. After connecting a custodial wallet -
  • Confirmed that after connecting custodian wallet, both the vBAT/external wallet balance is displayed on the Rewards Internals page
  • Confirmed that after connecting custodian wallet, total balance is displayed on the Rewards page, Rewards panel and Rewards NTP widget
Example Example Example Example
image image image image
4. Logged out state - PASSED
  • Confirmed that in the logged out state, the vBAT balance (zero) is shown only on the Rewards Internals page
  • Confirmed that Logged out UI is shown on the brave://rewards, BR panel, Rewards NTP and Tipping banner
  • Confirmed that : ( You are currently logged out text message is shown on the tipping banner
    • Note, tipping banner on Nightly 1.52.x will differ from 1.50.x and 1.51.x, this is because new design for tipping banner was merged to Nightly
Example Example Example Example Example
image image image image image
5. Fail the fetch balance request -
  • Gemini wallet (pre-req: have Charles Proxy set up to return 500 error for the endpoint (https://api.sandbox.gemini.com/v1/balances))
  • Confirmed that spinner animation is shown for both the Rewards BAT (vBAT) and external wallet balance on brave://rewards-internals page
Example Example Example Example
image image image image
6. Reset rewards - PASSED
  • Confirmed that after Reset rewards the wallet balance info is not displayed on the brave://rewards-internals page

image

bitFlyer - PASSED
1. Before enabling rewards - PASSED
  • Confirmed that there is no Balance info displayed on the brave://rewards-internals page before enabling the rewards
    image
2. After enabling rewards - PASSED
  • Confirmed that after enabling the rewards the vBAT balance is shown on the Rewards Internals page
  • Confirmed that the Connect account is shown on the brave://rewards settings page, BR panel and rewards NTP widget
Example Example Example Example
image image image image
3. After connecting a custodial wallet - PASSED
  • Confirmed that after connecting custodian wallet, both the vBAT/external wallet balance is displayed on the Rewards Internals page
  • Confirmed that after connecting custodian wallet, total balance is displayed on the Rewards page, Rewards panel and Rewards NTP widget
Example Example Example Example
image image image image
4. Logged out state - PASSED
  • Confirmed that in the logged out state, the vBAT balance (zero) is shown only on the Rewards Internals page
  • Confirmed that Logged out UI is shown on the brave://rewards, BR panel, Rewards NTP and Tipping banner
  • Confirmed that : ( You are currently logged out text message is shown on the tipping banner
    • Note, tipping banner on Nightly 1.52.x will differ from 1.50.x and 1.51.x, this is because new design for tipping banner was merged to Nightly
Example Example Example Example Example
image image image image image
5. Fail the fetch balance request - PASSED
  • bitflyer wallet (pre-req: have Charles Proxy set up to return "500 Internal Server Error" error for the endpoint (https://demo24kiuw4dcyncsy3qlud8u8.azurewebsites.net/api/link/v1/account/inventory)) - note, this differs from Uphold and Gemini sandbox environments where just returning 500 for the response status is enough
  • Confirmed that spinner animation is shown for both the Rewards BAT (vBAT) and external wallet balance on brave://rewards-internals page
Example Example Example Example
image image image image
6. Reset rewards - PASSED
  • Confirmed that after Reset rewards the wallet balance info is not displayed on the brave://rewards-internals page

image

@GeetaSarvadnya GeetaSarvadnya added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Brave Rewards
  
Done
Development

Successfully merging a pull request may close this issue.

6 participants