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

Improve Digital Bid Adapter: Added support for MultiBid #12777

Merged
merged 15 commits into from
Feb 19, 2025

Conversation

lyubomirshishkov
Copy link
Contributor

Type of change

  • Feature

Description of change

Added MultiBid support in Improve Digital's Bid Adapter.

lshishkov and others added 11 commits October 2, 2024 00:07
* **Type:** Fix
* **Scope:** improvedigitalBidAdapter
* **Subject:** Bid floors are always converted to USD.
* **Details:**
* Adds `DEFAULT_CURRENCY` variable which is set to USD
* Adds `convertBidFloorCurrency` function which in used to convert the bid floor when both `imp.bidfloor` and `imp.bidfloorcur` are present, and `imp.bidfloorcur` is not equal to the adapter's `DEFAULT_CURRENCY`;
* **Breaks:** N/A
…ed to USD;

* Removes rounding of bid floor when converting its currency to USD;
* remove redundant checks for type and NaN from `convertBidFloorCurrency` function
…urrency-convertion

Improve Digital Bid Adapter: Bid floor is sent in USD when possible
…id Adapter

* **Type:** Feature
* **Scope:** improvedigitalBidAdapter.js, improvedigitalBidAdapter_spec.js
* **Subject:** Adds multi-bid support in Improve Digital's Bid Adapter
* **Breaks:** N/A
…pport-to-improvedigitalBidAdapter

Improve Digital Bid Adapter: Added support for MultiBid
@patmmccann patmmccann self-assigned this Feb 18, 2025
@patmmccann
Copy link
Collaborator

can you document this choice in your adpter doc?

@lyubomirshishkov
Copy link
Contributor Author

@patmmccann patmmccann merged commit be1cee4 into prebid:master Feb 19, 2025
6 checks passed
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.

6 participants