-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 fails to upgrade segment database table from 1.19.x to 1.20.x and above #13719
Comments
@tmancey should this also be QA'd/verified on |
@kjozwiak Android testing would not hurt, however this is shared logic so desktop is good enough for me. Thanks |
Verified passed with
Verified STR from description and brave/brave-core#7678 (comment) Test Case 1
![]() ![]()
Test Case 2
![]() ![]() Verification passed on
Verified STR from description and brave/brave-core#7678 (comment) Test Case 1
Note: Needed to restart the browser to show ads Verified NTP ads are being counted. Example:
Verified ad notifications are being counted. Example:
Verified Encountered #13815 while testing Verification passed on
Test Case 2
Verified NTP ads are being counted. Example:
Verified ad notifications are being counted. Example:
Verified Verification passed on
Verified STR from description and brave/brave-core#7678 (comment) Test Case 1
Test Case 2
|
Description
Brave Ads fails to upgrade segment database table from 1.19.x to 1.20.x and above
Steps to Reproduce
Test Case 1:
Test Case 2:
Actual result:
Brave ads are not delivered due to failing to initialize ads
Expected result:
Brave ads should be delivered
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
No user data is lost, ads will work as expected after applying this fix.
The text was updated successfully, but these errors were encountered: