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

Added CCPA support and GDPR compliance to Cedato adapter #4683

Merged
merged 2 commits into from
Jan 3, 2020

Conversation

alexkh13
Copy link
Contributor

Type of change

  • Feature

Description of change

  1. Added CCPA support
  2. Removed client side cookie handling - for GDPR compliance
  3. Added wins count as part of the request

Other information

prebid/prebid.github.io#1724

Copy link
Contributor

@Fawke Fawke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @alexkh13,
Thanks for the PR. The change looks fine, but your code is laking in test-coverage, it's 73.68%, which is below our minimum acceptable level of 80%.

I also, I'd request you to submit a PR against 2.44.x-legacy branch as we're still actively maintaining that version.

@alexkh13
Copy link
Contributor Author

alexkh13 commented Jan 2, 2020

Hi @Fawke,
Thanks for the review. I've added some tests and created a PR for the legacy branch #4686

@alexkh13 alexkh13 requested a review from Fawke January 2, 2020 11:07
Copy link
Contributor

@Fawke Fawke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Fawke Fawke merged commit 598ceb5 into prebid:master Jan 3, 2020
redaguermas added a commit to redaguermas/Prebid.js that referenced this pull request Jan 8, 2020
…idVersion1.2.0

* 'master' of https://github.com/prebid/Prebid.js: (22 commits)
  fix lint errors in unit test file (prebid#4702)
  Add Revcontent Adapter (prebid#4654)
  Changed data structure in Platform One Analytic Adapter (prebid#4647)
  increment pre version
  Prebid 3.2.0 Release
  Add static API option to the consentManagementUsp module. (prebid#4685)
  replace all xhr stubs with global xhr stub to prevent all requests (prebid#4687)
  Add CCPA us_privacy support to spotxBidAdapter (prebid#4689)
  ucfunnel adapter support CCPA and remove utils.js in adapter (prebid#4541)
  freewheelSSPBidAdapter  (prebid#4645)
  Add CCPA support to Beachfront adapter (prebid#4673)
  add seedingAlliance Adapter (prebid#4614)
  Changed analytics data structure in YuktaMedia Analytic Adapter (prebid#4659)
  Add eplanning adapter for prebid 3.0 compliant and CCPA and GDPR support (prebid#4643)
  Bidder schain support (prebid#4551)
  Added CCPA support and GDPR compliance to Cedato adapter (prebid#4683)
  pass us privacy consent string to request (prebid#4581)
  Prebid 3 Admixer (prebid#4615)
  Pass uspConsent in bidRequest (prebid#4675)
  Advertly: New Bidder Adapter Submission (prebid#4496)
  ...
tadam75 pushed a commit to smartadserver/Prebid.js that referenced this pull request Jan 9, 2020
* cedato ccpa and gdpr compliance

* added gdpr usp and sync tests for cedato
@alexkh13 alexkh13 deleted the feature/cedato-ccpa branch February 20, 2020 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants