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 doesn't do anything #2106

Closed
junkguy opened this issue Nov 12, 2018 · 16 comments
Closed

Tipping doesn't do anything #2106

junkguy opened this issue Nov 12, 2018 · 16 comments
Labels
closed/invalid closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/rewards

Comments

@junkguy
Copy link

junkguy commented Nov 12, 2018

Description

Tipping lets you go through the process to send a tip, but wallet's BAT amount doesn't change and the tip is not listed in brave://rewards/

Steps to Reproduce

  1. In brave://rewards/, restore a wallet from a recovery key. The wallet has previous user grants, but is not self funded.
  2. On any site, click the rewards icon, click "Send A Tip...", select any amount of BAT, click "Send my tip"
  3. "Thank You" popup appears.
  4. Overall BAT amount is unchanged.

Actual result:

BAT amount is unchanged after tipping. On brave://rewards/, "Tips" section is empty

Expected result:

BAT amount should be deducted by the tipped amount.

Reproduces how often:

Always (when using a restored wallet. Haven't tried with a new wallet)

Brave version (brave://version info)

Brave 0.56.12 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    Don't know

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
    No

  • Is the issue reproducible on the latest version of Chrome?
    N/A

Additional Information

The restored wallet was from the Muon version of Brave.

@srirambv srirambv added this to the 1.x Backlog milestone Nov 13, 2018
@LaurenWags
Copy link
Member

Hey @junkguy - I'd like to get some additional information from you so the team can investigate but some of it is sensitive (it pertains to your wallet). Are you a member of our community (community.brave.com)? You can DM me there - I'm LaurenWags on community as well. Or if you're on Twitter, my handle is bravelaurenwags and we can DM there.

@Brave-Matt
Copy link

@eljuno
Copy link

eljuno commented Nov 15, 2018

@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Nov 16, 2018
@rebron
Copy link
Collaborator

rebron commented Nov 16, 2018

We think users from 55.22.x upgrading to 56.x are experiencing this issue. Would like to get this fixed for 0.57.x

@NejcZdovc
Copy link
Contributor

@kjozwiak @LaurenWags do we have STR for this one?

@NejcZdovc NejcZdovc added the needs-investigation A bug not 100% confirmed/fixed label Nov 19, 2018
@LaurenWags
Copy link
Member

@NejcZdovc I connected with the OP. seems like their wallets are now allowing them to tip. I'm wondering if at the time of attempting to tip there was a server issue which prevented them from doing so, which we've experienced before. So, it seems like we might be better served by putting in some kind of notification which lets users know that the server is unable to complete their tip request at present time. I did some looking and I'm wondering if this issue #1481 cover that scenario? cc @kjozwiak

@LaurenWags
Copy link
Member

Encountered this again today. We really need some kind of UI messaging to tell the user if their Tip didn't go thru and they should try again later. As it stands now, the only indication that your tip didn't go thru is that your balance doesn't go down and the tip isn't listed. If you're not watching those like a hawk you'll never know. A notification on the BAT logo would be helpful to let a user know that the tip failed. cc @rebron @kjozwiak

@NejcZdovc
Copy link
Contributor

@LaurenWags can you please paste ledger_state here, when this happens again. Thank you

@LaurenWags
Copy link
Member

@NejcZdovc doesn't that have potentially sensitive info on my wallet?

@NejcZdovc
Copy link
Contributor

@LaurenWags is this not staging testing wallet?

@LaurenWags
Copy link
Member

@NejcZdovc no, this was on Production.

@eljuno
Copy link

eljuno commented Dec 5, 2018

@kiwina
Copy link

kiwina commented Dec 11, 2018

PUT /v2/wallet/xxxxxxxxxxx
{"requestType":"httpSignature","signedTx":{"headers":{"digest":"SHA-256=xxxxxxxx","signature":"keyId=\"primary\",algorithm=\"ed25519\",headers=\"digest\",signature=\"xxxxxxxxxx\""},"body":{"denomination":{"amount":"1","currency":"BAT"},"destination":"xxxxxxxxxx"},"octets":"{\"denomination\":{\"amount\":\"1\",\"currency\":\"BAT\"},\"destination\":\"xxxxxxx\"}"},"surveyorId":"xxxxxxxx","viewingId":"xxxxxxx"}
{"statusCode":400,"error":"Bad Request","message":"Response Error: 400 Bad Request"}
happens on
Version 0.56.15 Chromium: 70.0.3538.110 (Official Build) (64-bit)
Version 0.57.18 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Version 0.59.2 Chromium: 71.0.3578.80 (Official Build) dev (64-bit)
tried several sites and asked several people all the same

@LaurenWags
Copy link
Member

Logged #2453 to improve error flow for tipping.

@Biglulux
Copy link

Biglulux commented Jan 12, 2019

I also have an issue where tipping doesn't work, where in some cases I don't get the option to tip at all for some websites. It seems to be random. The tip button also appears for some YouTubers, but not for others. Their verified status does not seem to make a difference. For example, I can tip GitHub (and GitHub is not verified) right now, but I cannot tip Google.
Version: Version 0.58.21 Chromium: 71.0.3578.98 (Official Build) (64-bit). Live release.

I am using a wallet that only has ever had user grants, never self funded, and not recovered.

@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@NejcZdovc NejcZdovc removed the priority/P2 A bad problem. We might uplift this to the next planned release. label May 8, 2019
@NejcZdovc
Copy link
Contributor

closing as stale

@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Apr 10, 2020
@NejcZdovc NejcZdovc removed the needs-investigation A bug not 100% confirmed/fixed label Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/invalid closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. feature/rewards
Projects
None yet
Development

No branches or pull requests

10 participants