Skip to content
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

Merged
merged 1 commit into from
Mar 5, 2020
Merged

vdo.ai bidder adapter #4814

merged 1 commit into from
Mar 5, 2020

Conversation

cpuBird
Copy link
Contributor

@cpuBird cpuBird commented Feb 3, 2020

Type of change

  • Bugfix
  • Feature
  • New 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

  • test parameters for validating bids
{
  bidder: 'vdo.ai',
  params: {
    placementId: 'newsdv77'
  }
}

@cpuBird
Copy link
Contributor Author

cpuBird commented Feb 11, 2020

Hey @GLStephen, Just wanted to hear your review on this pull request.

@GLStephen
Copy link
Collaborator

@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:
https://github.com/prebid/prebid-js-external-js-template#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.

@cpuBird
Copy link
Contributor Author

cpuBird commented Feb 17, 2020

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.

@GLStephen
Copy link
Collaborator

@cpuBird I think I had a few modules crossed up, will verify today

@GLStephen
Copy link
Collaborator

@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.

@cpuBird
Copy link
Contributor Author

cpuBird commented Feb 24, 2020

Hi @GLStephen,
Sorry for the late response. You can now test the adapter and it will give sample bids as expected.

@GLStephen
Copy link
Collaborator

@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.

@cpuBird
Copy link
Contributor Author

cpuBird commented Mar 3, 2020

@GLStephen Please test now as I've enabled mock bidder on that placement.

@GLStephen
Copy link
Collaborator

LGTM

@bretg
Copy link
Collaborator

bretg commented Mar 16, 2020

docs PR prebid/prebid.github.io#1841

rjvelicaria pushed a commit to openx/Prebid.js that referenced this pull request Apr 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants