Skip to content
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

Roll out Prebid with Permutive to 100% of audience #23824

Merged
merged 3 commits into from
May 19, 2021

Conversation

mxdvl
Copy link
Contributor

@mxdvl mxdvl commented May 19, 2021

What does this change?

Implements the Permutive changes in Prebid to 100% of the audience.

Follow-up on #23789, #23810 and #23811.
Relies on guardian/Prebid.js#117.

Does this change need to be reproduced in dotcom-rendering ?

  • No
  • Yes (please indicate your plans for DCR Implementation)

What is the value of this and can you measure success?

Better header bidding, more revenue from advertising partners.

Checklist

Does this change break ad-free?

  • No
  • It did, but tests caught it and I fixed it
  • It did, but there was no test coverage so I added that then fixed it

Tested

  • Locally
  • On CODE (optional)

@mxdvl mxdvl added commercial Commercial: Programmatic Changes to the implementation of the programmatic ad stack. labels May 19, 2021
@mxdvl mxdvl requested review from Amouzle and a team May 19, 2021 12:46
@mxdvl mxdvl force-pushed the prebid-with-permutive-rollout branch from d1d328a to 5402759 Compare May 19, 2021 12:56
@mxdvl mxdvl merged commit d2e3ffc into main May 19, 2021
@mxdvl mxdvl deleted the prebid-with-permutive-rollout branch May 19, 2021 13:06
@prout-bot prout-bot added Pending-on-PROD Seen-on-PROD and removed Commercial: Programmatic Changes to the implementation of the programmatic ad stack. commercial Pending-on-PROD labels May 19, 2021
@prout-bot
Copy link
Collaborator

Seen on PROD (merged by @mxdvl 13 minutes and 13 seconds ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants