-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Transition Brave Ads conversions queue from JSON to a database table #13697
Comments
Verification passed on
Verified test plan from: postview verificationpostviewNotification ad - same set
Verified conversion was successful:
Notification ad - same set - frequency capping
Notification ad - same set - browser restart
Verified conversion was successful
Notification ad - same set - observation window
Notification ad - same set - campaign removed from catalog
Notification ad - same set - expired campaign
Notification ad - same set - upgrade - trigger conversion
Notification ad - same set - upgrade - successful conversion
Notification ad - different set
Notification ad - different campaigns
NTP SI ad
Verified conversion was triggered:
Verified conversion was successful:
Verified conversion token was redeemed:
NTP SI ad - upgrade - trigger conversion
Verified conversion was triggered after upgrade:
NTP SI ad - upgrade - successful conversion
Verified ad was converted successfully after upgrade
Promoted Content ad
Verified conversion was triggered
Verified conversion was successful
postclick verification### postclickpostclick - notification ad - before click
postclick - notification ad - after click
Verified conversion was triggered:
postclick - notification ad - upgrade
postclick - NTP SI ad - before click
The conversion was triggered after a
postclick - NTP SI ad - after click
Verified conversion was triggered:
postclick - notification ad - upgrade
Verified conversion was triggered after upgrade:
postclick - promoted content - before click
Conversion was triggered. Logged #14795
postclick - promoted content - after click
Verified conversion was triggered:
postclick - promoted content - upgrade
Verified conversion was triggered after upgrade:
Verification passed on
Notification ad - different campaignsVerified conversion was triggered:
NTP SI adVerified conversion was triggered:
Verified conversion was successful:
NTP SI ad - upgrade - trigger conversionVerified conversion is triggered in an upgraded profile `1.22.64`
NTP SI ad - upgrade - successful conversionVerified conversion was triggered in 1.21.77
Verified conversion is successful in upgraded profile 1.22.64
Promoted Content adVerified conversion is triggered
Verified conversion is successful
postclick - notification ad - before clickAs per the test plan, the conversion should not be triggered but the AD conversion is triggered. Encountered #14795 (comment)
postclick - notification ad - after clickVerified conversion is triggered after clicking on the AD
postclick - notification ad - upgradeVerified conversion is triggered
postclick - NTP SI ad - before clickAs per the test plan, the conversion should not be triggered but the AD conversion is triggered. Encountered #14795 (comment)
postclick - NTP SI ad - after clickVerified conversion is triggered
postclick - notification ad - upgradeVerified conversion is triggered
Verified passed with
and
Notification ad - same setVerified test plan for Confirmed conversion triggered:
Confirmed conversion successful:
Notification ad - same set - upgrade - trigger conversionVerified test plan for Viewed ad with Confirmed conversion triggered after upgrade:
Notification ad - same set - upgrade - successful conversionVerified test plan for Viewed ad with Confirmed conversion successful:
NTP SI adVerified test plan for Confirmed conversion triggered:
Confirmed conversion successful:
Promoted Content adVerified test plan for Confirmed conversion triggered:
Confirmed conversion successful:
|
Transition Brave Ads conversions queue from JSON to a database table and add
campaign_id
andadvertiser_id
to unblock #13368 for @moritzhallerThe text was updated successfully, but these errors were encountered: