-
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
Instream and Outstream video support for medianetBidAdapter #5482
Instream and Outstream video support for medianetBidAdapter #5482
Conversation
….com/media-net/Prebid.js into instream-and-outstream-video-support
@bretg @Fawke @jaiminpanchal27 can you please review this PR and let me know your thoughts? |
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.
Thanks for the PR. I've gone through your changes and found few things you wanna look at:
-
Can you please remove all the references to the
adUnit.sizes
property from all the examples in the md file? I know, it's not related to this change, but, it can be confusing for someone new, since we've deprecatedadUnit.sizes
property from version 3 onwards. -
I tried to plug in this adUnit to get a response back, but getting an error. Can you check?
AdUnit:
[{
code: 'div-gpt-ad-1460505748561-0',
sizes: [640, 480],
mediaTypes: {
video: {
context: "instream",
playerSize: [640, 480]
}
},
bids: [{
bidder: 'medianet',
params: {
cid: '8CUX0H51C',
video: {
mimes: ['video/mp4']
}
}
}]
}];
I get the same error when I try outstream.
- Can you create a docs PR for prebid.org? Since you are adding support for a new mediaType, it can be listed under supported media types for your bid adapter.
@Fawke thanks for your feedback. I have incorporated all of the changes. Please have a look. |
Hi @c3p-0, Thanks for making the changes. But, still am not getting a vid bid response back from Also, we just release Prebid v4.0 few days back. You can read the blogpost for more details on the new changes. If you read the Moving Towards Standardization section, you'll notice that we are now enforcing moving video params, which I see you're currently passing under If @bretg or someone else approves the docs PR, then it's probably okay, no need to make the change, otherwise, you need to change the location of the video params. |
Hi, @Fawke with following AdUnit:
I am getting following response from https://prebid.media.net/rtb/prebid
sample curl call to https://prebid.media.net/rtb/prebid
Following is my sample test page
|
@c3p-0, Thanks for sharing the test page, am able to get a bid response. Last thing, just make a small modification to accept video params from both A publisher should be able to configure the adUnit like this as well:
|
@Fawke Thanks for your quick response. I have incorporated all of the changes. Please have a look. |
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.
@c3p-0 Thanks for making the changes.
Type of change
Description of change
Instream and Outstream video support for medianetBidAdapter