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

Tipping banner doesn't reflect failed vBAT tip - follow up to 19071 #24707

Closed
LaurenWags opened this issue Aug 16, 2022 · 3 comments
Closed

Tipping banner doesn't reflect failed vBAT tip - follow up to 19071 #24707

LaurenWags opened this issue Aug 16, 2022 · 3 comments
Assignees
Labels
closed/wontfix feature/rewards OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes

Comments

@LaurenWags
Copy link
Member

Description

Found while testing #19071

Case 1 (vBAT tip) from above issue does not show messaging for failed tip on the tip banner, instead banner shows success message and vBAT tip goes into endless retry loop.

Steps to Reproduce

Anon wallet (pre-req: have Charles Proxy set up to return 500 error for https://grant.rewards.bravesoftware.com/v1/suggestions)

  1. Clean profile, staging env
  2. Enable Rewards, claim UGP grant
  3. Go to tip a verified publisher
  4. Watch logs
  5. You will see the 500 error, but you get the success/confetti on tip banner

Actual result:

vBAT

Expected result:

Failed message or messaging indicating the tip will be retried to accurately reflect what is happening.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.43.71 Chromium: 104.0.5112.81 (Official Build) beta (x86_64)
Revision 5b7b76419d50f583022568b6764b630f6ddc9208-refs/branch-heads/5112@{#1309}
OS macOS Version 12.5 (Build 21G72)

Version/Channel Information:

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

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:

cc @Miyayes @zenparsing

@Miyayes
Copy link
Collaborator

Miyayes commented Aug 24, 2022

"it went into a perpetual retry state" — @LaurenWags

"in the vBAT case, if you get a 500 back from the endpoint, it just retries. So, the tipping banner thinks that it will succeed later, hence why it shows success." — @zenparsing

@Miyayes Miyayes added the priority/P3 The next thing for us to work on. It'll ride the trains. label Aug 24, 2022
@zenparsing
Copy link

zenparsing commented Aug 24, 2022

The non-trivial aspect here will be ensuring that the tokens are "unreserved" before failing the contribution. (The tokens are "reserved" before attempting to send them to the server.)

@Miyayes
Copy link
Collaborator

Miyayes commented Feb 22, 2023

Once vBAT is sunsetted completely, this will no longer be relevant. Will close as will not fix.

@Miyayes Miyayes closed this as not planned Won't fix, can't repro, duplicate, stale Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/wontfix feature/rewards OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes
Projects
None yet
Development

No branches or pull requests

3 participants