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

Still shows 7-day Ads History in popup linked from 30-day Ads History on brave://rewards #17063

Closed
stephendonner opened this issue Jul 21, 2021 · 4 comments · Fixed by brave/brave-core#9513
Assignees

Comments

@stephendonner
Copy link

stephendonner commented Jul 21, 2021

Description

Still shows 7-day Ads History in popup linked from 30-day Ads History on brave://rewards

(Cross-reference with #16627 which bumped # of ads history days from 7 -> 30.)

Steps to Reproduce

  1. new profile
  2. launch with --enable-logging=stderr --vmodule="*/variations/*"=6,"*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=3
  3. open a new-tab page
  4. click on Start using Rewards
  5. trigger, view, and click on an ad (either via opt-in to Brave News or regular)
  6. load brave://rewards
  7. look at the 30-days Ad History link
  8. click on it
  9. look at the # of days presented in the popup

Actual result:

30-day Ads History

Screen Shot 2021-07-20 at 6 08 17 PM

7-day ads history

Screen Shot 2021-07-20 at 6 19 03 PM

Expected result:

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.29.12 Chromium: 92.0.4515.101 (Official Build) nightly (x86_64)
Revision edb73f4fb624e2ea0cb6f5cc39c9e317ecd3535b-refs/branch-heads/4515@{#1536}
OS macOS Version 11.4 (Build 20F71)

Version/Channel Information:

  • Can you reproduce this issue with the current release? no
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the nightly channel? no
@stephendonner
Copy link
Author

Verified PASSED using the inline testplan with build

Brave 1.29.23 Chromium: 92.0.4515.107 (Official Build) nightly (x86_64)
Revision 87a818b10553a07434ea9e2b6dccf3cbe7895134-refs/branch-heads/4515@{#1634}
OS macOS Version 11.5 (Build 20G71)

Steps:

  1. new profile
  2. launch with --enable-logging=stderr --vmodule="*/variations/*"=6,"*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=3
  3. open a new-tab page
  4. click on Start using Rewards
  5. trigger, view, and click on an ad (either via opt-in to Brave News or regular)
  6. load brave://rewards
  7. look at the 30-days Ad History link
  8. click on it
  9. look at the # of days presented in the popup

Confirmed both places say 30 day(s) in the UI.

example example
Screen Shot 2021-07-23 at 9 50 30 AM Screen Shot 2021-07-23 at 9 50 34 AM

@stephendonner
Copy link
Author

@tmancey I think given #16627 we should take an uplift into 1.28.x for this issue; thoughts?

@tmancey
Copy link
Contributor

tmancey commented Jul 28, 2021

@stephendonner please see brave/brave-core#9543, thanks

@stephendonner
Copy link
Author

Verified FIXED on

Brave 1.28.96 Chromium: 92.0.4515.115 (Official Build) beta (x86_64)
Revision 48cb2f4029b84b003719740a6cf9ca73f374a857-refs/branch-heads/4515_105@{#4}
OS macOS Version 11.5.1 (Build 20G80)

Steps:

  1. new profile
  2. launch with --enable-logging=stderr --vmodule="*/variations/*"=6,"*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=9,"*/brave_user_model/*"=6 --brave-ads-staging --brave-ads-debug --rewards=staging=true,reconcile-interval=3
  3. open a new-tab page
  4. click on Start using Rewards
  5. trigger, view, and click on an ad (either via opt-in to Brave News or regular)
  6. load brave://rewards
  7. look at the 30-days Ad History link
  8. click on it
  9. look at the # of days presented in the popup
example example
Screen Shot 2021-07-29 at 11 32 54 AM Screen Shot 2021-07-29 at 11 32 58 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
3 participants