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

Feature/widespace update request param #1098

Merged

Conversation

mizmaar3
Copy link
Contributor

@mizmaar3 mizmaar3 commented Apr 3, 2017

Type of change

  • Code style update (formatting, local variables)

Description of change

  • Allow request script to get cached for faster bid response by breaking requestURL params with #
  • We have random number in query string in request url which was preventing cache our script in the middle
  • test parameters for validating bids
{
  bidder: 'widespace',
  params: {
    sid: '7b6589bf-95c8-4656-90b9-af9737bb9ad3',
    cur: 'EUR'
  }
}

Other information

@mkendall07 mkendall07 self-assigned this Apr 10, 2017
@mkendall07
Copy link
Member

LGTM

@matthewlane matthewlane merged commit b33a229 into prebid:master Apr 17, 2017
outoftime added a commit to Genius/Prebid.js that referenced this pull request Apr 19, 2017
…built

* 'master' of https://github.com/prebid/Prebid.js: (38 commits)
  Add optional domain parameter to AdButler adapter (prebid#1078)
  Send transactionID to Criteo Services (prebid#1113)
  Fix `buildMasterVideoTagFromAdserverTag()` not selecting winning bid (prebid#1106)
  Remove placement size selection and filtering (prebid#1107)
  revert `srcdoc` change (prebid#1130)
  Add new Adapter- Beachfront Media (prebid#1062)
  Fixes SpringServe adapter (prebid#1101)
  Update Widespace request param (prebid#1098)
  - New Adapter: Innity (prebid#1074)
  Update Roxot prebid analytic adapter (prebid#1034)
  Yarn Package Manager (prebid#1109)
  allow writing into current document if prebid is loaded inside an iframe (prebid#1066)
  Adapter bug fix (prebid#1096)
  fix typo
  added pr review process and governance model (prebid#1103)
  added support for sampling in ga and base adapter, fixed up some tests (prebid#1011)
  Add Inneractive adapter (prebid#1048)
  Add alias freewheel-ssp to stickyadstv bidder adapter  (prebid#1043)
  Add Facebook Audience Network adapter (prebid#1068)
  Add Atomx support (prebid#1056)
  ...
dluxemburg pushed a commit to Genius/Prebid.js that referenced this pull request Jul 17, 2018
* widespace adapter: separating params with # and addding hb, and hb.ver into it

* widespace adapter: update tests with new mandatory params

* widespace adapter: add adapter version number before # so we can force specific cached request script based on our adapter version

* widespace adapter: sid as query param

* widespace adapter: revert back sid after hash
@anniewalter anniewalter deleted the feature/Widespace_update_request_param branch February 20, 2019 09:57
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.

3 participants