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

One-time tip entries are temporarily not listed in brave://rewards #25751

Open
stephendonner opened this issue Oct 1, 2022 · 6 comments
Open
Assignees
Labels
blocked bug feature/rewards needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA/Yes regression release-notes/include

Comments

@stephendonner
Copy link

Description

One-time tip entries aren't listed in brave://rewards

Steps to Reproduce

  1. install 1.46.24
  2. launch Brave using Rewards/Ads staging
  3. click on the BAT icon on the URL bar
  4. click Start using Brave Rewards
  5. dismiss the next dialog
  6. click on Claim Tokens
  7. solve the CAPTCHA
  8. load https://laurenwags.github.io/
  9. click on the BAT icon
  10. click Send Tip
  11. click Send Tip
  12. open brave://rewards/
  13. note One-Time Tips reads -5.00 BAT in mini ledger summary
  14. scroll down to the Tips section above Monthly Tips
  15. look for your tip 👓

Actual result:

No tip(s) shown

example 1 example 2
Screen Shot 2022-09-30 at 9 57 30 PM Screen Shot 2022-09-30 at 9 57 44 PM

Expected result:

Tip(s) shown (listed out by recipient)

open brave://rewards-internals/ and note the tip

Screen Shot 2022-09-30 at 9 57 05 PM

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.46.24 Chromium: 106.0.5249.65 (Official Build) nightly (x86_64)
Revision 3269dc3633cdd2ab94546fdbe54962e45b17a6e0-refs/branch-heads/5249@{#580}
OS macOS Version 11.7 (Build 20G817)

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

cc @LaurenWags @GeetaSarvadnya @kjozwiak @zenparsing @Miyayes

@zenparsing
Copy link

I was able to reproduce on a local build. It appears that when tip records are queried from the database we filter them based on the month of the local time, but the dates are stored in UTC. This can cause issues for dates like 9/30; the date is stored in the DB as UTC but filtered using local time.

@kjozwiak
Copy link
Member

kjozwiak commented Oct 1, 2022

I was able to reproduce on a local build. It appears that when tip records are queried from the database we filter them based on the month of the local time, but the dates are stored in UTC. This can cause issues for dates like 9/30; the date is stored in the DB as UTC but filtered using local time.

Thanks @zenparsing. In that case, we can probably fix this whenever and remove as release/blocking. But I'll let the rewards team prioritise 👍 @LaurenWags @zenparsing @rebron please fee free to remove the release/blocking and prioritize as needed.

@GeetaSarvadnya
Copy link

It's working fine in all the channels for me. Is this macOS specific issue?

@LaurenWags
Copy link
Member

cc @Miyayes for prioritization

@Miyayes
Copy link
Collaborator

Miyayes commented Oct 3, 2022

I'm going to put a P2 on it, since this should probably be uplifted. That said, according to Kevin's investigation, it should only really rear its head at the end of the month (due to the different timezones), and isn't an across-the-board issue.

@Miyayes Miyayes added the priority/P2 A bad problem. We might uplift this to the next planned release. label Oct 3, 2022
@kjozwiak
Copy link
Member

kjozwiak commented Oct 3, 2022

So it looks like the Tip eventually does appear via brave://flags. Assuming it gets synced with sever-side and once the UTC/local time discrepancies gets resolved. Below is an example of the website that I tipped but didn't see it appear via brave:/flags due to the above. I checked a few hours ago and noticed it was now appearing/visible.

image

@rebron rebron removed this from the 1.44.x - Release #3 milestone Oct 4, 2022
@Miyayes Miyayes added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Oct 4, 2022
@Miyayes Miyayes changed the title One-time tip entries aren't listed in brave://rewards One-time tip entries are temporarily not listed in brave://rewards Oct 13, 2022
@Miyayes Miyayes added priority/P4 Planned work. We expect to get to it "soon". and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Apr 15, 2023
@Miyayes Miyayes added needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. blocked labels May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked bug feature/rewards needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA/Yes regression release-notes/include
Projects
None yet
Development

No branches or pull requests

9 participants