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

PBS Bid Adapter: outstream video renderer bugfix #7038

Merged
merged 4 commits into from
Jul 22, 2021
Merged

Conversation

ChrisHuie
Copy link
Collaborator

@ChrisHuie ChrisHuie commented Jun 16, 2021

From Issue #7037 . Checks that at least 1 renderer is speciified

@ChrisHuie ChrisHuie changed the title PBS Bid Adapter: outstream renderer bugfix PBS Bid Adapter: outstream video renderer bugfix Jun 16, 2021
Copy link
Collaborator

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Maybe a unit test to cover this?

modules/prebidServerBidAdapter/index.js Outdated Show resolved Hide resolved
Co-authored-by: Nepomuk Seiler <muuki88@users.noreply.github.com>
@ChrisHuie
Copy link
Collaborator Author

Maybe a unit test to cover this?

Good. Idea. I will add one today :)

@karimMourra
Copy link
Collaborator

LGTM

@patmmccann
Copy link
Collaborator

Covered in video taskforce today, thanks for the really important bugfix @ChrisHuie !

@ChrisHuie ChrisHuie merged commit ad14def into master Jul 22, 2021
@ChrisHuie ChrisHuie deleted the ChrisHuie-issueFix branch July 22, 2021 16:40
@patmmccann
Copy link
Collaborator

Take a look at #7208 for tests

prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
* PBS Bid Adapter: outstream renderer bugfix

* Update modules/prebidServerBidAdapter/index.js

Co-authored-by: Nepomuk Seiler <muuki88@users.noreply.github.com>

* revert to &&

* remove parens

Co-authored-by: Nepomuk Seiler <muuki88@users.noreply.github.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.

6 participants