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

Sortable Bid Adapter: update to comply with Prebid 5.x #7157

Merged
merged 2 commits into from Jul 9, 2021
Merged

Sortable Bid Adapter: update to comply with Prebid 5.x #7157

merged 2 commits into from Jul 9, 2021

Conversation

nlavallee-sortable
Copy link
Contributor

@nlavallee-sortable nlavallee-sortable commented Jul 8, 2021

Type of change

  • Other

Description of change

Updates to support new Prebid 5.x requirements.

  1. Add advertiserDomain support.
  2. Add price floors module support.
  3. Remove custom floor size map logic.

Other information

#5966 -> Prebid 5.0
#6650 -> 5.x issue for advertiserDomain
#6465 -> 5.x issue for floors module

The first commit re-introduces the bid adapter as it was in 4.x.

Use the price floors module.
Remove floor size map logic.
Add advertiserDomains to bid responses.
Update unit tests.
@nlavallee-sortable
Copy link
Contributor Author

I seem to be getting random build failures that are unrelated to my PR changes. I had tests failing, rebased, and now there's a different test failing. Can someone with permissions please re-run the build?

@ChrisHuie ChrisHuie self-requested a review July 9, 2021 12:00
@ChrisHuie ChrisHuie self-assigned this Jul 9, 2021
@ChrisHuie ChrisHuie changed the title Update Sortable bid adapter to comply with Prebid 5.x updates. Sortable Bid Adapter: update to comply with Prebid 5.x Jul 9, 2021
@ChrisHuie ChrisHuie merged commit cdea788 into prebid:master Jul 9, 2021
ChrisHuie added a commit to prebid/prebid.github.io that referenced this pull request Jul 19, 2021
bretg pushed a commit to prebid/prebid.github.io that referenced this pull request Jul 21, 2021
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