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

When balance cannot be fetched from custodian or user is in Logged out state, use unavailable text or "- -" for the Rewards balance #20991

Closed
3 tasks done
Miyayes opened this issue Feb 9, 2022 · 5 comments · Fixed by brave/brave-core#12839
Assignees
Labels

Comments

@Miyayes
Copy link
Collaborator

Miyayes commented Feb 9, 2022

Description

When a user is in a logged out state and can't query their balance from their custodian, their balance will usually show 0.0 BAT (since all vBAT has already been userdrained to the custodian).

  • Rewards panel
  • Rewards settings page
  • Rewards NTP widget

Update: This will also happen when custodian has downtime and the balance cannot be fetched. So, might be more general than when user is in a logged out state.

Solution

Use "N/A" (or some other string), which may be less alarming to users than seeing a declaration of 0.0 BAT.

@Miyayes Miyayes changed the title When user is in Logged out state, use something like "N/A" for the Rewards balance When user is in Logged out state, use unavailable text or "N/A" for the Rewards balance Feb 9, 2022
@Miyayes Miyayes added QA/Yes needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. labels Feb 9, 2022
@Miyayes Miyayes self-assigned this Feb 9, 2022
@Miyayes Miyayes added the priority/P4 Planned work. We expect to get to it "soon". label Feb 9, 2022
@Miyayes Miyayes changed the title When user is in Logged out state, use unavailable text or "N/A" for the Rewards balance When balance cannot be fetched from custodian or user is in Logged out state, use unavailable text or "N/A" for the Rewards balance Feb 10, 2022
@Miyayes Miyayes added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Mar 3, 2022
@Miyayes
Copy link
Collaborator Author

Miyayes commented Mar 3, 2022

Even if user is in a logged out state and receives some vBAT (an edgier case), I still think it's more beneficial to show N/A. The user can either disconnect from their custodian completely to get rid of N/A and see their vBAT there, or they can connect again and then the vBAT will be instantly userdrained to their custodial account.

@Miyayes Miyayes removed their assignment Mar 7, 2022
@zenparsing
Copy link

Related questions:

  • What happens on the tipping banner when the user is in this state? Do we disallow tipping (even if they have vBAT from an Ad grant)?
  • If we are displaying "N/A" and the user has vBAT, do we allow AC or monthly tips?

@Miyayes
Copy link
Collaborator Author

Miyayes commented Mar 11, 2022

This will help enforce correct vBAT vs. BAT balance states: #21588

@Miyayes
Copy link
Collaborator Author

Miyayes commented Apr 21, 2022

Using dashes instead of N/A, plus a little (i) tooltip.

See screenshots in the PR.

@Miyayes Miyayes changed the title When balance cannot be fetched from custodian or user is in Logged out state, use unavailable text or "N/A" for the Rewards balance When balance cannot be fetched from custodian or user is in Logged out state, use unavailable text or "- -" for the Rewards balance Apr 21, 2022
@zenparsing zenparsing added release-notes/include and removed needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. labels May 20, 2022
@zenparsing zenparsing added this to the 1.41.x - Nightly milestone May 20, 2022
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 27, 2022

Verification PASSED on


Brave | 1.41.77 Chromium: 103.0.5060.53 (Official Build) beta (64-bit)
-- | --
Revision | a1711811edd74ff1cf2150f36ffa3b0dae40b17f-refs/branch-heads/5060@{#853}
OS | Windows 10 Version 21H2 (Build 19044.1766)

Uphold_PASSED
  • Clean profile
  • Enable rewards
  • Link uphold account
  • Trigger Logged out-state via uphold->go to settings->application->authorization->remove brave
  • Confirmed that user can reconnect the Uphold accounts and see the account balance by clicking on the Log in to Gemini to see your balance via BR panel/settings page/NTP widget
Uphold Settings page BR panel NTP widget
image image image image
Gemini_PASSED
  • Clean profile
  • Enable rewards
  • Link uphold account
  • Trigger Logged out-state via Gemini Account->"Account"->Settings->Applications->You'll see Brave Browser listed. Hover over 3 dots menu and select "Revoke Access"
  • Confirmed that user can reconnect the Gemini accounts and see the account balance by clicking on the Log in to Gemini to see your balance via BR panel/settings page/NTP widget
Gemini Settings page BR panel NTP widget
image image image image
bitFlyer_PASSED
  • Clean profile
  • Enable rewards
  • Link uphold account
  • Trigger Logged out-state
  • Confirmed that the user can reconnect the bitFlyer account and see the account balance by clicking on the Log in to Gemini to see your balance via BR panel/settings page/NTP widget
bitFlyer Settings page BR panel NTP widget
image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants