-
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
pubmaticBidAdapter: battr in in banner requests #11917
Conversation
Sync up prebid master to fork master
…bid-master # Conflicts: # modules/pubmaticBidAdapter.js # test/spec/modules/pubmaticBidAdapter_spec.js
Sync with prebid master
Merge pull request #5 from pm-nitin-shirsat/master
Sync with master
Sync with master
Sync with prebid master
Sync with Prebidd.js master
… plcmt parameter.
…lcmt parameter, if it is missing.
Sync with prebid master
Sync with master
Sync with master
Sync with master
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.
You're getting this the wrong place, please support the standard location ortb2Imp
@patmmccann - Can you confirm that are you referring to read it from the below attached location? (i.e. adUnits -> ortb2Imp.banner.battr) |
You'll find it adjacent to this field if the publisher defines it that way Prebid.js/modules/pubmaticBidAdapter.js Line 772 in 74330a8
although i think ttd grabbed it in the wrong place Prebid.js/modules/ttdBidAdapter.js Line 244 in 7458aeb
I think it is .banner.battr? |
Thank you @patmmccann . I will update PR adding support for battr, in following location, ortb2Imp.banner.battr |
…id#11917) * Implement functionality for deal priority * Update test cases * kick off test manually * Added support of GPP to PubMatic adapter * gpp_sid in user syncs supposed to encode as a string, not an array * Remove extra space * Remove trailing spaces * Remove the placement parameter and update test cases accordingly, Add plcmt parameter. * Supporting placement parameter and logging warning message, for the plcmt parameter, if it is missing. * Remove commented code * Added plcmt in the pubmaticBidAdapter.md file * Adding support for Banner battr object * reading battr from ortb2Imp.banner --------- Co-authored-by: Chris Huie <[email protected]>
…id#11917) * Implement functionality for deal priority * Update test cases * kick off test manually * Added support of GPP to PubMatic adapter * gpp_sid in user syncs supposed to encode as a string, not an array * Remove extra space * Remove trailing spaces * Remove the placement parameter and update test cases accordingly, Add plcmt parameter. * Supporting placement parameter and logging warning message, for the plcmt parameter, if it is missing. * Remove commented code * Added plcmt in the pubmaticBidAdapter.md file * Adding support for Banner battr object * reading battr from ortb2Imp.banner --------- Co-authored-by: Chris Huie <[email protected]>
Type of change
Bugfix
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
Added support of battr params to pubmaticBidAdapter in banner requests
Other information