-
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
Update adkernelBidAdapter.js #5957
Conversation
description: Prebid adbite Bidder Adaptor Host: cpm.adbite.com
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 fine but add / update your tests for aliases.
They are failing
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.
We have change the array and count which you suggested. plz check and let us know if you need anything else.
Did you forget to check in and push to this branch? Still to associated change to the _spec file. I do not see a commit since my comment. |
@Avimobi I see you added two new PR's I have closed those and ask that you simply update this PR with the changes to the spec file please. |
Hello robert |
no worries. Thanks for making the update |
description: Prebid adbite Bidder Adaptor
Host: cpm.adbite.com
Type of change
Description of change
Be sure to test the integration with your adserver using the Hello World sample page.
For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:
Other information