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

Allowed setTargetingForGPTAsync() to target specific ad unit codes. #1158

Merged
merged 3 commits into from
May 18, 2017

Conversation

schiem
Copy link
Contributor

@schiem schiem commented Apr 26, 2017

Adds a paremeter to setTargetingForGPTAsync that allows it to set the
targeting for specific ad unit codes (as divs).

There is a bug where setTargetingForGPTAsync() is not accepting parameters, so issuing a call reset targeting for all available bids (GH-1129).

Testing has also been updated to reflect this.

Type of change

  • Bugfix

Other information

Adds a paremeter to setTargetingForGPTAsync that allows it to set the
targeting for specific ad unit codes (as divs).
@protonate protonate self-assigned this Apr 28, 2017
@protonate protonate modified the milestone: Prebid 0.23.0 May 2, 2017
Copy link
Collaborator

@protonate protonate left a comment

Choose a reason for hiding this comment

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

Please resolve conflicts.

@schiem
Copy link
Contributor Author

schiem commented May 17, 2017

@protonate Should be resolved now, it seems it was caused by a newline.

@protonate
Copy link
Collaborator

Great thanks, and now there are a few lint errors to clean up -- just need a space after comment double slash:

/home/travis/build/prebid/Prebid.js/src/prebid.js
  252:3  error  Expected space or tab after '//' in comment  spaced-comment
  255:3  error  Expected space or tab after '//' in comment  spaced-comment
  258:3  error  Expected space or tab after '//' in comment  spaced-comment
✖ 3 problems (3 errors, 0 warnings)

@schiem
Copy link
Contributor Author

schiem commented May 18, 2017

@protonate Updated, checks have now passed.

@protonate protonate merged commit 9c9bf4a into prebid:master May 18, 2017
outoftime pushed a commit to Genius/Prebid.js that referenced this pull request May 24, 2017
…built

* 'master' of https://github.com/prebid/Prebid.js: (23 commits)
  Increment pre version
  Probed 0.24.0 Release
  Beachfront adapter - add ad unit size (prebid#1183)
  Thoughtleadr adapter - fix postMessage (prebid#1207)
  When prebid server issues a no-bid response, call addBidResponse for every adUnit requested (prebid#1204)
  Improvement/timeout xhr (prebid#1172)
  Add native support (prebid#1072)
  Improvement/alias queue (prebid#1156)
  Updated documentaion (prebid#1160)
  Improvement/prebid iframes amp pages (prebid#1119)
  Fixes prebid#1114 possible xss issue (prebid#1186)
  Allowed setTargetingForGPTAsync() to target specific ad unit codes. (prebid#1158)
  updated tag (prebid#1212)
  Common user-sync (prebid#1144)
  Rename secureCreatives file and lint (prebid#1203)
  HIRO Media: Remove batching mechanism and use AJAX instead of JSONP (prebid#1133)
  Add Support for DigiTrust in Rubicon Adapter (prebid#1201)
  Upgrade linters to ESLint with stricter code style (prebid#1111)
  Add dynamic bidfloor parameter to Smart Adserver Adapter (prebid#1194)
  Bug fix: bids served by secure creatives does not get pushed into _winningBids (prebid#1192)
  ...
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request Jul 17, 2017
….23.0 to aolgithub-master

* commit '136fc37637749a764070c35c03e7e87a5c157947': (33 commits)
  Added changelog entry.
  Implemented passing key values feature.
  Update code to ESlint rules.
  Prebid 0.24.1 Release
  tests: drop ie9 browserstack test
  Audience Network: separate size from format (prebid#1218)
  Bugfix/target filtering api fix (prebid#1220)
  Map sponsor request param to endpoint param (prebid#1219)
  Increment pre version
  Probed 0.24.0 Release
  Beachfront adapter - add ad unit size (prebid#1183)
  Thoughtleadr adapter - fix postMessage (prebid#1207)
  When prebid server issues a no-bid response, call addBidResponse for every adUnit requested (prebid#1204)
  Improvement/timeout xhr (prebid#1172)
  Add native support (prebid#1072)
  Improvement/alias queue (prebid#1156)
  Updated documentaion (prebid#1160)
  Improvement/prebid iframes amp pages (prebid#1119)
  Fixes prebid#1114 possible xss issue (prebid#1186)
  Allowed setTargetingForGPTAsync() to target specific ad unit codes. (prebid#1158)
  ...
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.

2 participants