-
Notifications
You must be signed in to change notification settings - Fork 555
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
Set eager prebid test audience to 0% #26643
Conversation
package.json
Outdated
@@ -18,7 +18,7 @@ | |||
"@guardian/atom-renderer": "^2.0.0", | |||
"@guardian/automat-modules": "^0.3.8", | |||
"@guardian/braze-components": "^15.0.1", | |||
"@guardian/commercial": "11.15.0", | |||
"@guardian/commercial": "^11.16.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the addition of the caret intentional? I'm not against it's inclusion, just wanted to check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah good spot, no it wasn't. Will remove it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
0f62507
to
31a0842
Compare
066710d
to
280ad41
Compare
Seen on ADMIN-PROD (merged by @emma-imber 18 minutes and 12 seconds ago)
|
Seen on FRONTS-PROD (merged by @emma-imber 18 minutes and 20 seconds ago)
|
What does this change?
Bumps the version of the commercial bundle and sets the audience of the eager prebid test to 0%.
The test has now finished but we would like to retain the logic until we have the test results.