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

"Ad notifications received" should show Ads viewed between the 1st of the month and the 4th of the following month #3958

Closed
tmancey opened this issue Apr 1, 2019 · 3 comments · Fixed by brave/brave-core#2132

Comments

@tmancey
Copy link
Contributor

tmancey commented Apr 1, 2019

Description

"Ad notifications received" should show Ads viewed between the 1st of the month and the 4th of the following month which after further investigation is caused by the date range calculated in RewardsServiceImpl::GetEarningsRange()

Steps to Reproduce

  1. Set your devices date to the 6th of the month
  2. Launch the browser
  3. View several Ads
  4. Quit the browser
  5. Set your devices date to the 1st of the following month
  6. Launch the browser
  7. Visit brave://rewards

Actual result:

"Ad notifications received" does not match the amount of Ads viewed

Expected result:

"Ad notifications received" should match the amount of Ads viewed

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

0.63 and above

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? YES

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Additional Information

Date range is calculated in RewardsServiceImpl::GetEarningsRange()

@tmancey tmancey changed the title Incorrect date range for viewed ads notifications on the rewards panel "Ad notifications received" should show Ads viewed between the 1st of the month and the 4th of the following month Apr 1, 2019
@mandar-brave mandar-brave added the priority/P1 A very extremely bad problem. We might push a hotfix for it. label Apr 1, 2019
@mandar-brave
Copy link

@tmancey prioritized

@JoelMon
Copy link

JoelMon commented Apr 1, 2019

Happens to me with Version 0.64.30 Chromium: 73.0.3683.75 (Official Build) dev (64-bit).

The bug happens with me when I close and open the browser or when I turn off ads reward and turn it back on.
You can see the screen recording of the bug here sorry for the poor resolution.

@tmancey tmancey added this to the 0.65.x - Nightly milestone Apr 1, 2019
@tmancey tmancey self-assigned this Apr 2, 2019
tmancey added a commit to brave/brave-core that referenced this issue Apr 2, 2019
tmancey added a commit to brave/brave-core that referenced this issue Apr 2, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Apr 18, 2019

Verified passed with

Brave 0.63.45 Chromium: 74.0.3729.75 (Official Build) beta(64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Mac OS X
  • Verified STR from description. Verified ads viewed on the 6th were still showing on Ads panel on the 1st of following month. Verified when an Ad was viewed on the 1st it was added to the Ads panel appropriately.
Verification passed on Brave 0.63.45 Chromium: 74.0.3729.75 (Official Build) beta(64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Linux
  • Verified ads count retained when date is changed
  • Verified ads count increment when a new ad is seen/shown

Verification passed on

Brave 0.63.45 Chromium: 74.0.3729.75 (Official Build) beta (64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Windows 10 OS Build 17134.523

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment