-
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
Add Logicad for Publishers bid adapter #4010
Conversation
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.
Hi @logicad
Thanks for submitting this new adapter; I ran some tests and it appears to working as expected.
I had one question regarding the userSync code, please see below when you have the chance.
Thanks!
syncOptions.pixelEnabled && serverResponses[0].body.userSync.type == 'image') { | ||
return [{ | ||
type: 'image', | ||
url: serverResponses[0].body.userSync.url |
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.
To confirm - if you're returning multiple bids, this code would only grab the first bid's userSync pixel. The other bids' userSync pixels would be left out.
Is that expected?
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.
Hi @jsnellbaker
Thank you for the review.
Yes, it is as expected.
Only using the first bids will fulfill our requirements.
* Add Logicad for Publishers bid adapter * Add test code
* Add Logicad for Publishers bid adapter * Add test code
* Add Logicad for Publishers bid adapter * Add test code
Type of change
Description of change
New Logicad for Publishers bid adapter
contact email of the adapter’s maintainer: prebid@so-netmedia.jp
official adapter submission
A link to a PR on the docs repo at Logicad for Publishers bid adapter prebid.github.io#1395