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

HIRO Media: Remove placement size selection and filtering #1107

Merged
merged 1 commit into from
Apr 18, 2017

Conversation

ronenst
Copy link
Contributor

@ronenst ronenst commented Apr 6, 2017

Type of change

  • Other

Description of change

Size selection moved to server side.

Removed adapter parameters:

  • allowedSize
  • sizeTolerance

contact email of the adapter’s maintainer

support@hiro-media.com

  • official adapter submission

Other information

API change, will open a separate PR for documentation site.

Size selection moved to server side.

Changed API parameters accordingly:
`selectedSize` will simply be the first size in the sizes array
`placementSizes` -> `additionalSizes` and includes sizes besides the first one

Removed adapter parameters:
- allowedSize
- sizeTolerance
ronenst pushed a commit to hiromedia/prebid.github.io that referenced this pull request Apr 6, 2017
Relevant PR on main project repo: prebid/Prebid.js#1107
@mkendall07
Copy link
Member

LGTM

@mkendall07 mkendall07 merged commit 2a45a16 into prebid:master Apr 18, 2017
outoftime added a commit to Genius/Prebid.js that referenced this pull request Apr 19, 2017
…built

* 'master' of https://github.com/prebid/Prebid.js: (38 commits)
  Add optional domain parameter to AdButler adapter (prebid#1078)
  Send transactionID to Criteo Services (prebid#1113)
  Fix `buildMasterVideoTagFromAdserverTag()` not selecting winning bid (prebid#1106)
  Remove placement size selection and filtering (prebid#1107)
  revert `srcdoc` change (prebid#1130)
  Add new Adapter- Beachfront Media (prebid#1062)
  Fixes SpringServe adapter (prebid#1101)
  Update Widespace request param (prebid#1098)
  - New Adapter: Innity (prebid#1074)
  Update Roxot prebid analytic adapter (prebid#1034)
  Yarn Package Manager (prebid#1109)
  allow writing into current document if prebid is loaded inside an iframe (prebid#1066)
  Adapter bug fix (prebid#1096)
  fix typo
  added pr review process and governance model (prebid#1103)
  added support for sampling in ga and base adapter, fixed up some tests (prebid#1011)
  Add Inneractive adapter (prebid#1048)
  Add alias freewheel-ssp to stickyadstv bidder adapter  (prebid#1043)
  Add Facebook Audience Network adapter (prebid#1068)
  Add Atomx support (prebid#1056)
  ...
rmloveland pushed a commit to prebid/prebid.github.io that referenced this pull request May 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants