-
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
vdo.ai bidder adapter #4814
vdo.ai bidder adapter #4814
Conversation
Hey @GLStephen, Just wanted to hear your review on this pull request. |
@cpuBird I believe the manner in which you are loading external scripts is in violation of the external script policy. Can you explain what you are trying to do with the loadExternalScript call? The Policy: Also, your sample params are not returning bids with which to validate the adapter is working. Correct the second and provide some info on the first and we'll reevaluate. |
Hi @GLStephen , I can see that we haven't used loadExternalScript call anywhere in our code. Can you send me any screenshot of the code where you have seen it. And also I've fixed the sample params from our end. |
@cpuBird I think I had a few modules crossed up, will verify today |
@cpuBird I'm still getting a noBid from your end, given that it's a new adapter I'd like to verify that it works before approving it. I determined it was another adapter raising the external script warning. |
Hi @GLStephen, |
@cpuBird I am still not seeing bids with your test params and we cannot accept the bidder without functional test params, this is for myself to ensure quality as well as future users to ensure they have params they can use to test their own installs. |
@GLStephen Please test now as I've enabled mock bidder on that placement. |
LGTM |
docs PR prebid/prebid.github.io#1841 |
Type of change
Description of change