-
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
ConnectID: Support new easy-opt out method #9069
ConnectID: Support new easy-opt out method #9069
Conversation
@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 🙏 |
@ChrisHuie - I'm not sure what I was thinking by submitting this without additional tests. I've included some now + updated the branch. |
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 |
* 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]>
* 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]>
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