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

OpenX adpater: adding CCPA support #4584

Merged
merged 8 commits into from
Dec 18, 2019
Merged

Conversation

jimee02
Copy link
Contributor

@jimee02 jimee02 commented Dec 11, 2019

Type of change

  • Feature

Description of change

Adding CCPA support

Other information

Documentation update: prebid/prebid.github.io#1676

bwschmidt and others added 3 commits December 10, 2019 13:38
So that we can track CCPA compatible versions of our adapter.
OpenX: Updated adapter version number
@harpere harpere self-requested a review December 12, 2019 07:23
@harpere harpere self-assigned this Dec 12, 2019
@harpere
Copy link
Collaborator

harpere commented Dec 12, 2019

you should add us_privacy to your user sync request also

@bwschmidt
Copy link
Contributor

you should add us_privacy to your user sync request also

@harpere does it look okay now?

@harpere
Copy link
Collaborator

harpere commented Dec 18, 2019

@bwschmidt looks good, though you should also open a PR against the 2.44.x-legacy branch for those who are not upgrading to 3.0 right away.

@harpere harpere merged commit 6b9b230 into prebid:master Dec 18, 2019
sa1omon pushed a commit to CleanMediaNet/Prebid.js that referenced this pull request Dec 19, 2019
* OpenX adpater: adding CCPA support

* OpenX: Updated adapter version number

So that we can track CCPA compatible versions of our adapter.

* OpenX adpater: adding CCPA support

* OpenX: Updated adapter version number

So that we can track CCPA compatible versions of our adapter.

* OpenX Adapter: added gdpr and us privacy data to user sync in the event
the sync goes out before the response

* fix lint and broken unit tests
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.

3 participants