-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Conversant Bid Adapter: handle paapi data in the response #11663
Conversation
modules/conversantBidAdapter.js
Outdated
@@ -176,7 +176,18 @@ export const spec = { | |||
* @return {Bid[]} An array of bids which were nested inside the server. | |||
*/ | |||
interpretResponse: function(serverResponse, bidRequest) { | |||
return converter.fromORTB({request: bidRequest.data, response: serverResponse.body}); | |||
const ortbBids = converter.fromORTB({request: bidRequest.data, response: serverResponse.body}); | |||
const ortbAuctionConfigs = deepAccess(serverResponse, 'body.ext.igi') || []; |
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.
is this not handled here now?
0ee4efd#diff-24ee01f9467dc2a1704da10ca938150df2b3386d6b5b732870760dbc7cd1c4d7R373
cc @dgirardi
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.
looks like as of today it is. I'll go back through my changes and check it out
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.
yes - this is no longer necessary. note that if you decide to keep it, paapiAuctionConfigs
was renamed to just paapi
(since it can also contain igs
now) .
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.
trying to validate if ortbConverter handles out of the box now since you've used the proper formatting in your response...
afb846c
to
9945b9a
Compare
9945b9a
to
7c06ec6
Compare
Type of change
Bugfix
[X ] Feature
New bidder adapter
Updated 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
Conversant bid adapter: Add support for handling paapi data in bid response
Other information