-
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
Add gdpr and targeting support for Yieldlab adapter #2755
Conversation
Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit.
Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
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.
one small question, otherwise LGTM
}) | ||
|
||
if (bidderRequest && bidderRequest.gdprConsent) { | ||
query.consent = bidderRequest.gdprConsent.consentString | ||
query.gdpr = (typeof bidderRequest.gdprConsent.gdprApplies === 'boolean') ? bidderRequest.gdprConsent.gdprApplies : true |
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.
just verifying that you want to default gdprApplies
to true
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.
Yes, gdprApplies=true should be the default. Thanks for making sure!
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
* Add targeting support Support for optional key-value targeting. As the request to Yieldlab is joint, we expect the targeting params to be present for only one of the adUnits or to be the same for every adUnit. * Add GDPR support Support for the Prebid.js consent management module. Passing "consent" and "gdpr" (default: true) parameters if present.
Type of change
Add gdpr support and key-value targeting to the Yieldlab bidder prebid.github.io#851
Description of change
solutions@yieldlab.de