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

AdPlayerPro Video Module : initial module release #12150

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

artemAdp
Copy link
Contributor

@artemAdp artemAdp commented Aug 20, 2024

Type of change

  • Feature

Description of change

New video module for AdPlayer.Pro

Other information

@ChrisHuie ChrisHuie changed the title Video Module: Add new video module for AdPlayer.Pro AdPlayer.Pro Aug 20, 2024
@ChrisHuie ChrisHuie changed the title AdPlayer.Pro AdPlayerPro Video Module : initial module release Aug 20, 2024
Comment on lines +117 to +118
function getOrtbContent() {
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

If the player has content information such as media source url , media id, media title etc, it would be good to populate. Contextual targeting can help get higher bids

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This module only supports outstream format, so it does not use content.

Copy link
Collaborator

@karimMourra karimMourra left a comment

Choose a reason for hiding this comment

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

Overall looks great, just one small observation

modules/adplayerproVideoProvider.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@karimMourra karimMourra left a comment

Choose a reason for hiding this comment

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

@ChrisHuie LGTM!

@ChrisHuie
Copy link
Collaborator

@artemAdp can you please add documentation for this new module in our docs repo https://github.com/prebid/prebid.github.io

@artemAdp
Copy link
Contributor Author

@ChrisHuie Thank you for your comment. I have added documentation.

@ChrisHuie ChrisHuie merged commit 323bc37 into prebid:master Oct 2, 2024
6 checks passed
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