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

IX Bid Adapter : reading video placement property #6994

Merged
merged 2 commits into from
Jun 16, 2021

Conversation

punkiller
Copy link
Contributor

Type of change

  • Feature

Description of change

To read placement property from video params.
If thats not present, set a default placement

  • official adapter submission

Other information

@punkiller punkiller marked this pull request as ready for review June 14, 2021 20:21
@punkiller punkiller changed the title Ix placement IX Bid Adapter : reading video placement property Jun 14, 2021
Copy link
Member

@mkendall07 mkendall07 left a comment

Choose a reason for hiding this comment

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

LGTM

@mkendall07
Copy link
Member

build failed, restarting.
@punkiller if you want this to be supported in 4.x please also open a PR against that branch https://github.com/prebid/Prebid.js/tree/4.43.x-legacy

@mkendall07 mkendall07 self-assigned this Jun 16, 2021
@mkendall07 mkendall07 merged commit c3b96a9 into prebid:master Jun 16, 2021
AlvaroBrey pushed a commit to Parrable/Prebid.js that referenced this pull request Jun 17, 2021
* master: (103 commits)
  Opt Out Bid Adapter: add new bid adapter (prebid#7029)
  Dependencies/Testing: update dependencies, npm audit fixes, and bump Browserstack versions (prebid#6828)
  5.2.0-pre
  Prebid.js 5.1.0
  IX Bid Adapter :  reading video `placement` property (prebid#6994)
  changing PBS debug flag to boolean (prebid#7035)
  Ats analytics set sampling rate to 1 percent (prebid#7010)
  Update sspBC bid adapter (v5.0) (prebid#7002)
  Gamma Bid Adapter: support adomain for Prebid 5.0 (prebid#7033)
  Datablocks Bid Adapter : update bid adapter and add 5.0 compliance (prebid#6765)
  Add microadBidAdapter (prebid#7007)
  Adf Bid Adapter: add adform alias (prebid#7009)
  Fixed FPD issue after for Prebid.js 5.0 (prebid#7031)
  Zeta Ssp Bid Adapter: provide devicetype (prebid#7026)
  make sizes a variable as the code expects (prebid#7024)
  VidazooBidAdapter: support for response meta.advertiserDomains (v5.0) (prebid#7022)
  add uid2 to OpenX Bid Adapter (prebid#7015)
  Zeta Ssp Bid Adapter: page and domain fields from config (prebid#7019)
  tappx Bid Adapter: fix wrong requests with undefined params (prebid#7021)
  RealVu Analytics Adapter: remove "tablet" from device types (prebid#7016)
  ...
@punkiller
Copy link
Contributor Author

Thanks for restarting the build and the suggestion @mkendall07 , I just got back from vacation, so I will create a PR against the legacy branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants