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

ConnectID: Support new easy-opt out method #9069

Merged

Conversation

slimkrazy
Copy link
Contributor

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

ConnectID is utilised by other client-side libraries outside of Prebid, and Yahoo has introduced a new mechanisms that allow them to provide their users with another way to opt out. This change ensure that our ConnectID userID module respects this.

Other information

Documentation PR: prebid/prebid.github.io#4055

@ChrisHuie
Copy link
Collaborator

@slimkrazy can you please add some unit test coverage and pull in recent commits to master. This pr is pretty far behind and looks like acting weird with es-lint in our CircleCi testing 🙏

@slimkrazy
Copy link
Contributor Author

@ChrisHuie - I'm not sure what I was thinking by submitting this without additional tests.

I've included some now + updated the branch.

@slimkrazy
Copy link
Contributor Author

Hey @ChrisHuie - may I ask what more I need to do here? Seems the documentation PR was already merged and released:

https://docs.prebid.org/dev-docs/modules/userId.html#honoring-privacy-choices

@ChrisHuie ChrisHuie self-requested a review October 11, 2022 14:32
@ChrisHuie ChrisHuie self-assigned this Oct 11, 2022
@ChrisHuie ChrisHuie merged commit 958acae into prebid:master Oct 11, 2022
JacobKlein26 pushed a commit to nextmillenniummedia/Prebid.js that referenced this pull request Feb 9, 2023
* Respect custom opt-out override localstorage key.

* Wrap opt-out check in a function.

* Rename function

* Add unit tests

Co-authored-by: slimkrazy <[email protected]>
jorgeluisrocha pushed a commit to jwplayer/Prebid.js that referenced this pull request May 23, 2023
* Respect custom opt-out override localstorage key.

* Wrap opt-out check in a function.

* Rename function

* Add unit tests

Co-authored-by: slimkrazy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants