-
Notifications
You must be signed in to change notification settings - Fork 0
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
Sovrn Adapter accept AdUnitCodes #11
Conversation
…-gpp EX-4658 Update Prebid Server user sync for GPP
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.
LGTM
continue | ||
} | ||
|
||
imp.Ext = impExtBuffer |
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.
Do we really want to send whole bidder object in the ext
object of the imp?
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.
That's the bidder object I was trying to completely get rid of in my prebid-server-java PR, about which you had questions and bad feelings. So as I left it untouched in prebid-server-java, here I did the same. So now both go and java repos are in sync. Do you have any recommendations on this?
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.
One question
No description provided.