-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Try to trigger NTP SI on each NTP refresh #24107
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
Comments
@btlechowski, this is intentional, as we show prefetched NTP SI ads. So on each 4th NTP refresh, we do the following:
|
The above requires |
Verification passed on
ads enabledVerified attempt to serve the ad is made every NTP load 1st NTP - ad is served (prefetched):
2nd NTP - ad is viewed (shown):
3rd NTP - attempt to serve the ad is made but blocked due frequency capping
4th NTP - attempt to serve the ad is made but blocked due frequency capping
5th NTP - ad served
6th NTP - ad viewed
7th NTP - ad is served
8th, 9th - ad is already served so we do nothing 10th and is viewed
ads disabledVerified ads are shown but user is not paid 1st ad not shown |
NTP SI is shown every other trigger
Steps to Reproduce
Actual result:
On 1st trigger the logs shown that NTP SI is served, but in reality it is not shown.
1st NTP SI trigger
2st NTP SI trigger, NTP SI is viewed and actually shown to the user
Expected result:
NTP SI is shown when expected
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
Version/Channel Information:
cc @rebron @tmancey @jsecretan @brave/qa-team
The text was updated successfully, but these errors were encountered: