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

Smartadserver Bid Adapter: Update default value of startDelay video parameter #11225

Merged
merged 35 commits into from Apr 3, 2024

Conversation

krzysztofequativ
Copy link
Contributor

Type of change

[x] Feature

Description of change

The startDelay parameter is set to 1 by default.

mcourouble and others added 30 commits November 10, 2022 10:39
Support additional video params
}
}
return 2;// Default value for all exotic cases set to bid.params.video.startDelay midroll hence 2.
return 1; // SADR-5619
Copy link
Collaborator

Choose a reason for hiding this comment

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

this appears wrong; what do you do if videoMediaType.startdelay is say, 180? In this case, you appear to ignore the declaration and guess that it is preroll?
Also, you seem to have dropped the .startdelay from your switch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, @patmmccann, I've updated the implementation. Thx.

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 publisher specifies midroll starting at 3 minutes (180), you are calling it preroll still?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are completely right. It's fixed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @patmmccann, could you review it, please? Thx.

@patmmccann patmmccann merged commit 74a404c into prebid:master Apr 3, 2024
2 of 3 checks passed
mefjush pushed a commit to adhese/Prebid.js that referenced this pull request Apr 8, 2024
…arameter (prebid#11225)

* Smartadserver Bid Adapter: Add support for SDA user and site

* Smartadserver Bid Adapter: Fix SDA support getConfig and add to unit testing

* support floors per media type

* Add GPP support

* Rework payloads enriching

* Add gpid support

* Support additional video params

* vpmt as array of numbers

* Fix comment

* Update default startDelay

* Include videoMediaType's startdelay

* Handle specified midroll

---------

Co-authored-by: Meven Courouble <mcourouble@smartadserver.com>
Co-authored-by: Krzysztof Sokół <88041828+smart-adserver@users.noreply.github.com>
Co-authored-by: Dariusz O <darekodz@gmail.com>
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

5 participants