Problem/Motivation

The header bidding for sourcepoint isn't set up quite right and the DFP integration needs some alterations in line with Sourcepoint documentation.

Example of header bidding:
https://documentation.sourcepoint.com/cmp/configurations/cmp-for-prebid-js

Example of DFP integration:
https://documentation.sourcepoint.com/cmp/configurations/strategies-to-m...

Proposed resolution

  • Make sure the cmp_shim script is the first thing on the page so it is above the header bidding.
  • Follow the instructions from sourcepoint to correctly implement the DFP firing for consent.
CommentFileSizeAuthor
#7 3124491-header-bidding-7.patch1.96 KBtashaharrison80
#5 3124491-header-bidding-5.patch688 bytestashaharrison80
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tashaharrison80 created an issue. See original summary.

tashaharrison80’s picture

tashaharrison80 CreditAttribution: tashaharrison80 at Dennis commented
Title: Better support for heading bidding » Better support for header bidding

  • tashaharrison80 committed 4798448 on 7.x-2.x-3124491-header-bidding-dfp 
    Issue #3124491 cmp_shiv.js always first

  • tashaharrison80 committed 81882ec on 7.x-2.x-3124491-header-bidding-dfp 
    Issue #3124491 weight isnt so dramatic
tashaharrison80’s picture

tashaharrison80 CreditAttribution: tashaharrison80 at Dennis commented
FileSize
3124491-header-bidding-5.patch688 bytes

  • tashaharrison80 committed 96581a3 on 7.x-2.x-3124491-header-bidding-dfp 
    Issue #3124491 add waitForConsentData option
tashaharrison80’s picture

tashaharrison80 CreditAttribution: tashaharrison80 at Dennis commented
FileSize
3124491-header-bidding-7.patch1.96 KB

New patch to add support for waitForConsentData

tashaharrison80’s picture

tashaharrison80 CreditAttribution: tashaharrison80 at Dennis commented
Status: Active » Fixed

Merged and tagged.

tashaharrison80’s picture

tashaharrison80 CreditAttribution: tashaharrison80 at Dennis commented
Status: Fixed » Closed (fixed)