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 : remove razr creative logic #12678

Merged
merged 9 commits into from
Jan 22, 2025

Conversation

cciocov
Copy link
Contributor

@cciocov cciocov commented Jan 21, 2025

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Updated bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

We removed some code/logic concerning RAZR creatives, which are no longer used. No functional or API changes.

lshishkov and others added 8 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
@ChrisHuie ChrisHuie changed the title Improvedigital/remove razr Improve Digital Bid Adapter : remove razr creative logic Jan 22, 2025
@patmmccann patmmccann self-assigned this Jan 22, 2025
@patmmccann patmmccann merged commit e5bf921 into prebid:master Jan 22, 2025
5 of 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.

4 participants