-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
CRITEO RTUS Integration with Yieldmo Prebid #5075
Conversation
Upstream sync
This reverts commit 4fd8634.
pass along gdpr consent string and gdpr applies through get
add stringified schain
change parameter names
updates bidder module to be 3.0 compliant, and tests
Fs 7766 remove openxoustream
…erer object in bidderRequest
remove utils functions that will be depricated in 3.0 and use new ref…
…pliant ad units object
use prebids gettopwindow util functions. update markdown with 3.0 com…
Pass along us_privacy string
This reverts commit 4fd8634.
…erer object in bidderRequest
…pliant ad units object
Rebase upstream
CRITEO RTUS For prebid
This reverts commit 0185407.
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.
Hi @mglowackiym,
The change looks fine to me. I've tested with the hello world page on prebid and getting a response back. But when I checked your response headers, I don't see this key Content-Encoding
. Ideally, it should be set to gzip
. Can you please add it to the server response header?
@Fawke we made the update to our ad response. It should be now |
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.
LGTM
* update maintainer email * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * pass along gdpr consent string and gdpr applies through get * use paramter gdprConsent * add stringified schain * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * deletes openxoutstream adapter * fix broken yieldmo test * remove log * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * remove blank.min * Fix unit tests and remove unused variables and methods * add stringified schain * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * pass along gdpr consent string and gdpr applies through get * use paramter gdprConsent * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * deletes openxoutstream adapter * fix broken yieldmo test * remove log * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * update maintainer email * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * add stringified schain * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * fix broken yieldmo test * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * Clean up console * Do not undo my featureeee * CRITEO RTUS For prebid * Tests for CRITEO RTUS For prebid * Revert "Do not undo my featureeee" This reverts commit 0185407. * linting * add test back in: Co-authored-by: Melody Li <melodyli@ip-192-168-2-119.ec2.internal> Co-authored-by: Elber Carneiro <elber@yieldmo.com> Co-authored-by: Elber Carneiro <elberdev@gmail.com> Co-authored-by: Melody Li <melody@yieldmo.com> Co-authored-by: Melody Li <ml867@cornell.edu> Co-authored-by: Andrew Holz <holz.andrew@gmail.com> Co-authored-by: Michal <mike@lamiglo.com>
* 'master' of https://github.com/prebid/Prebid.js: (102 commits) Marsmedia - Add vastXml and fix id response (prebid#5067) PubMatic adapter to support image sync (prebid#5104) minor consentManagement fix (prebid#5050) fix circle ci failing tests (prebid#5113) Add Relaido Adapter (prebid#5101) Add new bid adapter for ConnectAd (prebid#4806) change payload (prebid#5105) Utils updates (prebid#5092) Read OpenRTB app objects if set in config + bug fix for when ad units are reloaded (prebid#5086) Criteo : added first party data mapping to bidder request (prebid#4954) updateAdGenerationManual (prebid#5032) New bid adapter: Wipes (prebid#5051) Prebid manager analytics utm tags (prebid#4998) CRITEO RTUS Integration with Yieldmo Prebid (prebid#5075) isSafariBrowser update (prebid#5077) Support min &max duration for onevideo (prebid#5079) increment pre version Prebid 3.15.0 release prebid#5011 Fix to set Secure attribute on cookie when SameSite=none (prebid#5064) Prebid adapter for windtalker (prebid#5040) ...
* update maintainer email * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * pass along gdpr consent string and gdpr applies through get * use paramter gdprConsent * add stringified schain * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * deletes openxoutstream adapter * fix broken yieldmo test * remove log * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * remove blank.min * Fix unit tests and remove unused variables and methods * add stringified schain * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * pass along gdpr consent string and gdpr applies through get * use paramter gdprConsent * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * deletes openxoutstream adapter * fix broken yieldmo test * remove log * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * update maintainer email * add stringified schain * Revert "add stringified schain" This reverts commit 4fd8634. * add stringified schain * don't change other files * change parameter namves * updates bidder module to be 3.0 compliant, and tests * fix broken yieldmo test * remove utils functions that will be depricated in 3.0 and use new referer object in bidderRequest * use prebids gettopwindow util functions. update markdown with 3.0 compliant ad units object * Pass along us_privacy string * Clean up console * Do not undo my featureeee * CRITEO RTUS For prebid * Tests for CRITEO RTUS For prebid * Revert "Do not undo my featureeee" This reverts commit 0185407. * linting * add test back in: Co-authored-by: Melody Li <melodyli@ip-192-168-2-119.ec2.internal> Co-authored-by: Elber Carneiro <elber@yieldmo.com> Co-authored-by: Elber Carneiro <elberdev@gmail.com> Co-authored-by: Melody Li <melody@yieldmo.com> Co-authored-by: Melody Li <ml867@cornell.edu> Co-authored-by: Andrew Holz <holz.andrew@gmail.com> Co-authored-by: Michal <mike@lamiglo.com>
Type of change
Description of change
This PR contains a new feature which is an integration of Criteo with Yieldmo's prebid.
The implementation follows guidelines described here and has been tested successfully both locally and by our internal testing team. Moreover, in this PR a new test has been added to cover the new use case. Lastly, Prebid linting rules have been applied as part of this. Please note that test placement for this adapter only renders on mobile.
Other information
This change has been verified locally and by internal testing team. Feature testing and internal PR review has been completed under this PR. Please note that test placement for this adapter only renders on mobile.