Skip to content

PulsePoint Lite adpater changes#1630

Merged
matthewlane merged 22 commits intoprebid:masterfrom
pulsepointinc:ET-2642
Oct 3, 2017
Merged

PulsePoint Lite adpater changes#1630
matthewlane merged 22 commits intoprebid:masterfrom
pulsepointinc:ET-2642

Conversation

@anand-venkatraman
Copy link
Contributor

Type of change

  • Feature

Description of change

Updates to the adapter to prepare for 1.0 upgrade.
Support for InApp parameters.

Test Page: http://projects.contextweb.com/av/prebid/1.0-pre/

!!(bid && bid.params && bid.params.cp && bid.params.ct)
),

buildRequests: bidRequest => {
Copy link
Contributor Author

@anand-venkatraman anand-venkatraman Sep 27, 2017

Choose a reason for hiding this comment

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

Can you please confirm the first param of this method is the array of bid requests? It appears so based on the call made from here. But this contradicts with the information here, where bidderRequest param is an object, with a property bids, holding the bid requests.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for pointing this out Anand. Opened a PR (prebid/prebid.github.io#380) on prebid.org to correct the docs. @snapwich will submit a PR to clarify the comments in the code.

Note that we also opted to rename the first parameter because there're actually two params formerly called "bidRequests" and "bidderRequest". This was kinda confusing, so we renamed the first to "validBidRequests". The second param isn't documented on prebid.org as it's expected to be needed only in rare cases, but is documented in the code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for clarifying @bretg

import adaptermanager from 'src/adaptermanager';
import { registerBidder } from 'src/adapters/bidderFactory';

const NATIVE_DEFAULTS = {
Copy link
Contributor

Choose a reason for hiding this comment

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

These could go into native.js if they are per the spec (i think they are).

Doesn't need to change for merge though.

/**
* Attempts to capture the referrer url.
*/
function referrer() {
Copy link
Contributor

Choose a reason for hiding this comment

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

this should also be a generic function (todo for utils).

@matthewlane matthewlane merged commit 36f4ba7 into prebid:master Oct 3, 2017
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request Oct 25, 2017
….30.0 to aolgithub-master

* commit '5a8d2bf93ee15071a78e24ac976103cacf3c6021': (35 commits)
  Added changelog entry.
  Prebid 0.30.1 Release
  Remove undefined variable usage (prebid#1662)
  fixes bug for IE when invalid value passed to parse (prebid#1657)
  Aliasbidder fix (prebid#1652)
  prebidAdapter secure support (prebid#1655)
  Increment pre version
  Prebid 0.30.0 Release
  Add native param support to mediaTypes (prebid#1625)
  PulsePoint Lite adpater changes (prebid#1630)
  Appnexus ast unittest updates (prebid#1654)
  Support aspect ratio specification for native images (prebid#1634)
  Revert changes for switch between client side and server side. (prebid#1653)
  rubicon converted to bidderFactory (prebid#1624)
  Add JSDoc for `pbjs.getAllWinningBids` (prebid#1566)
  Add ignore-loader to handle .md files (prebid#1646)
  fixed PBS cookie syncs (prebid#1637)
  Add placementId request param to Yieldmo bid adapter (prebid#1632)
  Adxcg analytics adapter (prebid#1599)
  Add publisher sub-id support to the Criteo adapter (prebid#1629)
  ...
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.

4 participants

Comments