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

Sharethrough adapter to use ajax #1042

Merged
merged 1 commit into from Mar 21, 2017
Merged

Sharethrough adapter to use ajax #1042

merged 1 commit into from Mar 21, 2017

Conversation

rizhang
Copy link
Contributor

@rizhang rizhang commented Mar 9, 2017

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

update sharethrough adapter to use ajax for bid requests instead of iframes in order to speed up bid response times.

  • test parameters for validating bids
    N/A
  • contact email of the adapter’s maintainer
    rzhang@sharethrough.com
  • official adapter submission

Other information

update sharethrough adapter to use ajax for bid requests
Copy link
Member

@mkendall07 mkendall07 left a comment

Choose a reason for hiding this comment

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

LGTM. Receiving valid bids via Ajax. Nice improvement, also fixes the undefined body error.

@mkendall07 mkendall07 assigned mkendall07 and unassigned protonate Mar 21, 2017
@mkendall07 mkendall07 requested a review from protonate March 21, 2017 18:19
@mkendall07
Copy link
Member

@protonate

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.

Looks good, tests green, approved.

@protonate protonate merged commit fb094cd into prebid:master Mar 21, 2017
@rizhang
Copy link
Contributor Author

rizhang commented Mar 22, 2017

#1069 We added another PR because we realized we weren't sending cookies to our bidder endpoint.

vzhukovsky pushed a commit to aol/Prebid.js that referenced this pull request Apr 29, 2017
update sharethrough adapter to use ajax for bid requests
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request May 6, 2017
…nc-with-aol-github to aolgithub-master

* commit '0c6c83cc2bfaeeafa9da967c90d83da4af58e7b3':
  Add change log entry.
  Renaming and fix missing semicolon.
  Update parsing script tag regexp.
  Refactoring. Replace stub bids objects, add additional case for rendering pixels.
  Refactoring. Replace test consts by methods.
  Create separate namespace for aol globals, add unit tests.
  Fix unit tests, remove unnecessary spaces.
  Implement render pixels only once feature.
  Add change log entry.
  Add reporting ID for analytics.
  Add Facebook Audience Network adapter
  fix specs
  update version
  fix jslint errors
  Change to using a closure for the callback from ajax to preserve bidObj
  Ensure cookies get sent in request headers (prebid#1069)
  Rz/ajax (#5) (prebid#1042)
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request May 6, 2017
…MP-157-sharethrough-adapter to release/1.16.0

* commit '0ee6e1af865fd9ab7d7e07dbcf4b52e9b6d0c91d':
  Add creative key in bid response for Sharethrough adapter.
  fix specs
  update version
  fix jslint errors
  Change to using a closure for the callback from ajax to preserve bidObj
  Ensure cookies get sent in request headers (prebid#1069)
  Rz/ajax (#5) (prebid#1042)
vzhukovsky added a commit to aol/Prebid.js that referenced this pull request May 6, 2017
…16.0 to master

* commit '56ce8e64d179bb0913a3b471e19e9650849b16b2':
  Add change log entry.
  Add creative key in bid response for Sharethrough adapter.
  Add reporting ID for analytics.
  Add Facebook Audience Network adapter
  fix specs
  update version
  fix jslint errors
  Change to using a closure for the callback from ajax to preserve bidObj
  Ensure cookies get sent in request headers (prebid#1069)
  Rz/ajax (#5) (prebid#1042)
vzhukovsky pushed a commit to aol/Prebid.js that referenced this pull request May 22, 2017
update sharethrough adapter to use ajax for bid requests
dluxemburg pushed a commit to Genius/Prebid.js that referenced this pull request Jul 17, 2018
update sharethrough adapter to use ajax for bid requests
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