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

Dailymotion Bid Adapter: accept ortb2 field #11366

Merged
merged 39 commits into from May 7, 2024

Conversation

sebmil-daily
Copy link
Contributor

@sebmil-daily sebmil-daily commented Apr 19, 2024

Type of change

Description of change

This addresses the comments that we received after #10970 was merged.

The main changes are:

  • use standard ORTB video parameters as fallback
  • support several new ORTB video fields
  • better request validation to exclude non video/instream requests
  • preliminary support for in-app requests

Kevin Siow and others added 30 commits January 22, 2024 17:22
Dailymotion Bid Adaptor: initial release
The same way as we can have an array of IAB categories level 1 in the ORTB request, this PR introduces an array for the IAB categories level 2.

To be forward compatible with level [2.2](https://github.com/InteractiveAdvertisingBureau/Taxonomies/blob/main/Content%20Taxonomies/Content%20Taxonomy%202.2.tsv) and [3.0](https://github.com/InteractiveAdvertisingBureau/Taxonomies/blob/main/Content%20Taxonomies/Content%20Taxonomy%203.0.tsv) specifications, the category IDs should be sent as strings.
feat(LEO-528): Allow multiple IAB categories level 2 in video metadata
Dailymotion bid adapter: Clarify the video metadata to provide in each player context
Co-authored-by: Rumesh <rumesh@users.noreply.github.com>
…to_bidparam

Dailymotion bid adapter: Move API key to bid params (fix tests)
Dailymotion Bid Adaptor: add iabcat1 and fallback to ortb2 for iabcat2
…ocument mapping of ortb2 fpd to video metadata
Kevin Siow and others added 6 commits April 19, 2024 15:45
Note: I changed the case of `startdelay` to be consistent with the other parameters, but it won't have any impact on current deployments as this parameter is not used onsite.
…deo_fields

Dailymotion Bid Adaptor: Add standard ORTB video parameters
# Conflicts:
#	modules/dailymotionBidAdapter.js
#	modules/dailymotionBidAdapter.md
#	test/spec/modules/dailymotionBidAdapter_spec.js
@sebmil-daily
Copy link
Contributor Author

sebmil-daily commented Apr 19, 2024

Sorry for the long list of commits coming from the upstream merge in our fork. We'll rebase it as soon as this one is merged.

cc. @patmmccann and @karimMourra, thanks again for your time and comments, we are open to any new suggestion :)

@patmmccann patmmccann changed the title Dailymotion Bid Adapter leftovers Dailymotion Bid Adapter: accept ortb2 field Apr 19, 2024
@@ -38,17 +45,22 @@ function getVideoMetadata(bidRequest, bidderRequest) {
const videoMetadata = {
description: videoParams.description || '',
duration: videoParams.duration || 0,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
duration: videoParams.duration || 0,
duration: videoParams.duration || contentObj.len || 0,

Per spec: len - integer - Length of content in seconds; appropriate for video or audio

modules/dailymotionBidAdapter.js Show resolved Hide resolved
modules/dailymotionBidAdapter.md Outdated Show resolved Hide resolved
modules/dailymotionBidAdapter.md Show resolved Hide resolved
@patmmccann patmmccann removed their request for review April 19, 2024 20:34
@patmmccann patmmccann removed their assignment Apr 19, 2024
@sebmil-daily
Copy link
Contributor Author

Thanks @karimMourra for all the suggestions. I think @kvnsw adressed all of them, could you confirm please ?

@patmmccann
Copy link
Collaborator

LGTM

@karimMourra if you have additional feedback let's handle in subsequent pr's

@karimMourra
Copy link
Collaborator

Apologies, I lost track of this PR. The changes look great thank you for addressing!
Swing by the Video Taskforce sync sometime! We meet every 2 weeks on Thursdays at 12PM EST. We are meeting this week.

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

6 participants