Skip to content

Fix bug with supporting non-cpm dealId bids.#1006

Merged
mkendall07 merged 3 commits intomasterfrom
bugfix/support-deal-non-cpm-bids
Feb 26, 2017
Merged

Fix bug with supporting non-cpm dealId bids.#1006
mkendall07 merged 3 commits intomasterfrom
bugfix/support-deal-non-cpm-bids

Conversation

@mkendall07
Copy link
Contributor

Type of change

  • Bugfix
  • Feature
  • New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other

Description of change

Support non-cpm dealId bids. Including support for Xaxis adapter.

Other information

@prebid/core for review

@daniel-hoffmann
Copy link
Contributor

@mkendall07 Hey! Looks good to me in regards to the changes in the adapter.

@mkendall07
Copy link
Contributor Author

@snapwich
want to give this a quick look as I think you refactored some of this code before?

@snapwich
Copy link
Collaborator

snapwich commented Feb 24, 2017

LGTM. 👍

With that change the deal should come through automatically as hb_deal if you want to use alwaysUseBid or enableSendAllBids rather than manually passing in adapter with custom bidderSettings.

@mkendall07
Copy link
Contributor Author

@snapwich thanks. Yeah actually I don't think hb_deal comes through because it's considered a default key now, which is omitted on non-winning bids because we don't want to collide with the winning bid.

@mkendall07 mkendall07 merged commit eea9914 into master Feb 26, 2017
@matthewlane matthewlane deleted the bugfix/support-deal-non-cpm-bids branch March 2, 2017 17:44
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request May 19, 2017
…18.0 to aolgithub-master

* commit '2881f03dd76bfcf484fbfeaf0f9e1000b383d20d':
  Added adapters in aolPartnersIds.json.
  Added changelog entry.
  Prebid 0.20.0 Release
  Remove test that intermittently fails unit tests (prebid#1020)
  Create bids for requests without mutating ad units (prebid#1019)
  Use optimize js to improve parse time (https://github.com/nolanlawson/optimize-js) (prebid#1018)
  Fixes for Centro adapter (prebid#992)
  adbund adapter (prebid#932)
  Fix bug with supporting non-cpm dealId bids. (prebid#1006)
  Added referrer parameter (prebid#966)
  Support Video for GetIntent adapter (prebid#987)
  Currency support for SmartAdServer adapter (prebid#979)
  Deal support for SmartAdServer adapter (prebid#978)
  Skip test causing build failures (prebid#997)
  E2etest concurrency (prebid#982)
  Add ThoughtLeadr adapter (prebid#894)
  Increment Pre Version
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.

3 participants

Comments