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

Brave Ads may not be converted for duration of observation window #12234

Closed
tmancey opened this issue Oct 21, 2020 · 2 comments
Closed

Brave Ads may not be converted for duration of observation window #12234

tmancey opened this issue Oct 21, 2020 · 2 comments
Assignees
Labels
bug closed/duplicate Issue has already been reported feature/ads OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Win64 QA/Yes release-notes/include

Comments

@tmancey
Copy link
Contributor

tmancey commented Oct 21, 2020

Description

Brave Ads may not be converted for duration of observation window depending on user activity and ads history.

Steps to Reproduce

  1. View an ad notification (which has a conversion)
  2. Wait for the ad notification to time out
  3. Trigger 700 thumbs up events on the 7-days ad history screen (use staging so you do not run out of unblinded tokens)
  4. Trigger the conversion from step 1

Actual result:

Ad is not converted due to the ads history only storing 700 entries (enough for ~7 days depending on how many events are triggered)

Expected result:

Ad should be converted for the duration of the observation window

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

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? N/A
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

@tmancey tmancey added bug priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include feature/ads OS/Desktop labels Oct 21, 2020
@tmancey tmancey self-assigned this Oct 21, 2020
@tmancey tmancey changed the title Brave Ads are not converted for observation windows greater than ~14 days Brave Ads may not be converted for duration of observation window Oct 21, 2020
@tmancey tmancey added the closed/duplicate Issue has already been reported label Nov 3, 2020
@tmancey
Copy link
Contributor Author

tmancey commented Nov 3, 2020

Closing as issue was resolved by refactoring #12486 where we moved ad event history from client.json to an ad_events database table

@tmancey tmancey closed this as completed Nov 3, 2020
@btlechowski
Copy link

Verification passed on

Brave 1.18.66 Chromium: 87.0.4280.67 (Official Build) dev (64-bit)
Revision 0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS Windows 7 Service Pack 1 (Build 7601.24544)

Verified test plan from the description.
Verified that after 700 events on 7-day history page, the ad was converted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug closed/duplicate Issue has already been reported feature/ads OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Win64 QA/Yes release-notes/include
Projects
Archived in project
Development

No branches or pull requests

2 participants