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

OR Bid Adapter: add support for advertiserDomains and floors module #6890

Merged
merged 46 commits into from
Jun 1, 2021
Merged

OR Bid Adapter: add support for advertiserDomains and floors module #6890

merged 46 commits into from
Jun 1, 2021

Conversation

hendrikiseke1979
Copy link
Contributor

Type of change

  • Feature

Description of change

To keep up with upcoming changes for prebid v5, this PR will:

  • use the getBidFloor-Module
  • fill the meta object with advertisementDomains, if sent by ssp

RainerVolk4014 and others added 30 commits November 11, 2019 11:46
orbidder adapter: add withCredentials:true header to BidRequest and o…
adding extra line to trigger build ... again
@patmmccann patmmccann changed the title add compliance prebid 5.0 OR Bid Adapter: add support for advertiserDomains and floors module May 31, 2021
@ChrisHuie ChrisHuie self-requested a review June 1, 2021 13:01
@ChrisHuie ChrisHuie self-assigned this Jun 1, 2021
@ChrisHuie ChrisHuie merged commit d7a1418 into prebid:master Jun 1, 2021
prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
…rebid#6890)

* orbidder adapter: add withCredentials:true header to BidRequest and onBidWon Requests

* add blank in order to trigger build again

* remove blank to trigger build ... again

* adding extra line to trigger build ... again

* add prebid version to request

* add unit test for version parameter

* add version parameter to win requests

* fix comment

* trigger rebuild

* trigger rebuild

* remove onBidWon callback from adapter

* fix retrieving orbidder endpoint url from local storage

* fix unit tests

* use getBidFloor function for prebidv5 compatibility

* fill meta.advertiserDomains from serverResponse.adomain for prebidv5 compatibility

* drop eslint-disable

* switch misleading expect and actual in unit test

* cr: rename adomain to advertiserDomains

* trigger build pipeline

* trigger build pipeline

Co-authored-by: Volk, Rainer <Rainer.Volk@otto.de>
Co-authored-by: RainerVolk4014 <53347752+RainerVolk4014@users.noreply.github.com>
Co-authored-by: siggi-otto <57615762+siggi-otto@users.noreply.github.com>
Co-authored-by: Hendrik Iseke <39734979+hiseke@users.noreply.github.com>
Co-authored-by: Hendrik Iseke <hendrik.iseke@gmx.de>
Co-authored-by: rvolk <>
Co-authored-by: Arne Schulz <Arne.Schulz@otto.de>
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.

5 participants