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

add bigRichmedia adapter #3558

Merged

Conversation

mediaconsortium-develop
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Feb 9, 2022

✔️ Deploy Preview for prebid-docs-preview ready!

🔨 Explore the source changes: 2f5f368

🔍 Inspect the deploy log: https://app.netlify.com/sites/prebid-docs-preview/deploys/6204e015f990b500081238d1

😎 Browse the preview: https://deploy-preview-3558--prebid-docs-preview.netlify.app

dev-docs/bidders/bigRichmedia.md Outdated Show resolved Hide resolved
dev-docs/bidders/bigRichmedia.md Outdated Show resolved Hide resolved
| `video` | optional | Object containing video targeting parameters. See [Video Object](#appnexus-video-object) for details. | `video: { playback_method: ['auto_play_sound_off'] }` | `object` |

#### Video Object

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a note saying that these params should be placed in mediaTypes.video rather than as bidder-specific params.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

remove option
add a note for params
Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the response from Xandr on using their GVLID "Actually, since they are using our SSP tech, it means we would be the controller of the data processed in the auction. So at the very least, our global vendor id should be used. Then depending on additional data processing they may do on their own, they may also need their own vendor id."

@fowler446 fowler446 merged commit 7c93682 into prebid:master Feb 16, 2022
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.

None yet

4 participants