-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
ads resources are not loaded after browser restart #16918
Labels
feature/ads
OS/Desktop
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Test-Plan-Specified
QA/Yes
regression
release/blocking
release-notes/exclude
Milestone
Comments
btlechowski
changed the title
Follow up to #16307 - conversionEnvelope is not attached to conversion's confirmation request
ads resources are not loaded after browser restart
Jul 13, 2021
24 tasks
tmancey
added
priority/P2
A bad problem. We might uplift this to the next planned release.
and removed
needs-investigation
A bug not 100% confirmed/fixed
labels
Jul 13, 2021
4 tasks
Verification passed on
First run
2nd run
3rd run
Verification passed on
2nd run
3rd Run
Verified passed with
First Run
Second Run
Third Run
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/ads
OS/Desktop
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Test-All-Platforms
QA/Test-Plan-Specified
QA/Yes
regression
release/blocking
release-notes/exclude
After browser restart, the ads resources will not load. Purchase intent, anti targeting and verifiable conversion will not work
This is a regression introduced with Brave News Ads #16008 (confirmed with @tmancey and @iambrianfung in a meeting)
Steps to Reproduce
Actual result:
ads resources never initialize
Expected result:
ads resources are initialized
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
Not reproducible on
Reproducible on
cc @tmancey @moritzhaller @iambrianfung @jsecretan @brave/legacy_qa @rebron
The text was updated successfully, but these errors were encountered: