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

Audience Network: allow native/fullwidth bids for non-IAB sizes #2203

Merged

Conversation

lovell
Copy link
Contributor

@lovell lovell commented Feb 28, 2018

Type of change

  • Bugfix

Description of change

Hello, this change removes IAB size validation from the Audience Network adaptor for "native" bids and ensures "fullwidth" bids are 300x250, as discussed in #2196.

New unit tests for the isBidRequestValid() and buildRequests() functions that cover these scenarios are included.

This work was commissioned and paid for by Facebook.

Update: added "fullwidth" size check.

@lovell lovell force-pushed the audience-network-native-allow-non-iab-sizes branch from 36ab8b6 to 38de40a Compare March 1, 2018 23:36
Copy link
Collaborator

@harpere harpere left a comment

Choose a reason for hiding this comment

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

LGTM

@harpere harpere added the LGTM label Mar 6, 2018
@harpere harpere merged commit c54a179 into prebid:master Mar 6, 2018
@lovell lovell deleted the audience-network-native-allow-non-iab-sizes branch March 7, 2018 09:22
mizmaar3 added a commit to widespace-os/Prebid.js that referenced this pull request Mar 8, 2018
* master: (27 commits)
  Increment pre version
  Prebid 1.5.0 Release
  Fix cross-platform test failures (prebid#2228)
  Fix uncahced video bids from multi-response array triggering callback early (prebid#2219)
  Add vuble adapter (prebid#2201)
  Update Vidazoo domain (prebid#2223)
  InSkin Bid Adapter: remove referrer field from request body (prebid#2217)
  Gamma Support UserSync Endpoint (prebid#2216)
  Feature/stale bot (prebid#2192)
  33Across Bid Adapter: updated user sync endpoint (prebid#2193)
  Adding PR_REVIEW guideline (prebid#2159)
  Add FairTrade Bid Adapter (prebid#2147)
  Add banner support to Beachfront adapter (prebid#2117)
  Smartyads Adapter 1.x (prebid#2080)
  Audience Network: allow native bids for non-IAB sizes (prebid#2203)
  Update position default value in rubicon (prebid#2214)
  Auctionmanager spec refactor pr (prebid#2155)
  fix mediaType being overwritten by undefined in rubicon bid adapter (prebid#2209)
  Fix lint error (prebid#2208)
  VAST support in adform adapter (prebid#2173)
  ...
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