-
Notifications
You must be signed in to change notification settings - Fork 50
Handle cases for opted out emails #944
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
Handle cases for opted out emails #944
Conversation
| var pbjs = pbjs || {}; | ||
| pbjs.que = pbjs.que || []; | ||
| pbjs.que.push(function () { | ||
| setUid2Config(); |
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.
Removed to prevent initialziation conflicts + keep consistent with EUID
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.
i think we actually want to keep this, and add it to the EUID site. Good eye looking for consistency though, this case would just be adding it to the other site instead.
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.
Most users would want to set the config early when loading the site, not late like EUID does it now
Currently when users enter an opted out email in the prebid sample site, nothing happens. Updated functionality so user knows whats happening similar to how EUID prebid sample sites displays an opted out message.