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

Support aspect ratio specification for native images #1634

Merged
merged 1 commit into from
Oct 3, 2017

Conversation

matthewlane
Copy link
Collaborator

@matthewlane matthewlane commented Sep 28, 2017

Type of change

  • Feature

Description of change

This change refactors the native mapping portion of appnexusAst adapter to allow specifying aspect ratios for native image and icon assets. Aspect ratios are defined within a native image parameter with the keyword aspect_ratios, and a ratio_width and ratio_height. min_width may optionally be given. Example:

mediaType: 'native',
nativeParams: {
  title: {required: true},
  body: {required: true},
  image: {
    required: true,
    aspect_ratios: [{
      min_width: 2000,
      ratio_width: 588,
      ratio_height: 379,
    }]
  },
}

@matthewlane matthewlane force-pushed the native-aspect-ratio branch 2 times, most recently from c368889 to be9940d Compare September 28, 2017 18:24
},
sponsoredBy: 'sponsored_by'
sponsoredBy: 'sponsored_by',
Copy link
Member

Choose a reason for hiding this comment

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

are trailing commas allowed now?? 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jaiminpanchal27 jaiminpanchal27 merged commit aef474d into master Oct 3, 2017
@matthewlane matthewlane deleted the native-aspect-ratio branch October 3, 2017 19:30
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)
  ...
dluxemburg pushed a commit to Genius/Prebid.js that referenced this pull request Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants