Platform.io video support

[varashellov]

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

Added support for video

• test parameters for validating bids

  {
    bidder: 'platformio',
    params: {
            pubId: '29521',
            siteId: '26049',
            size: '640X480',
            placementId: '123',
    }
  }

• contact email of the adapter’s maintainer
• official adapter submission

[varashellov]

Docs prebid/prebid.github.io#688

[jsnellbaker]

Hello @varashellov,
The changes for the video support seem good. However I was running into some trouble when I was retesting the banner and native ad units.

For the banner ad unit, I was seeing a CORS related error message in the console (which is a bit strange since it was only this ad unit that gave this error).

For the native ad unit, I was seeing the POST request to your server go out but nothing came back in the response. The POST completed as a 204.

I was using the hello_world as my test page and I used the testing information in the md file. Would you be able to look into the banner & native? Thanks.

[varashellov]

Hello @jsnellbaker
Thanks for the quick response.
Please check hello_world page https://github.com/varashellov/Prebid.js/blob/master/integrationExamples/gpt/hello_world.html

[jsnellbaker]

Hi @varashellov, thanks for providing the sample page. I found the issues I made during the banner and native testing. The ads are delivering fine now.

Can you back out the change to the hello_world.html page in this PR? We should be good to merge once that's done.

[varashellov]

@jsnellbaker
Uploaded the default file
Thanks

[jsnellbaker]

@varashellov Can you just remove the file from the PR entirely? That would be better than uploading the same version of the file. Thanks.

[jsnellbaker]

Hi @varashellov The last commit actually deletes the file instead of removing it from the PR. I checked internally with some other folks, and they actually recommended to just re-add the file back into the PR at this point since it would be the easiest way to fix it.

When you have the chance, can you please add the master version of this file to the PR?

Thanks, and sorry for the back and forth.