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

ImRtdProvider: add imuid param #12475

Conversation

eknis
Copy link
Contributor

@eknis eknis commented Nov 20, 2024

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

  • add imuid param

Other information

@@ -80,7 +80,25 @@ export function getBidderFunction(bidderName) {
);
}
return bid
}
},
craft: function (bid, data, moduleConfig) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the special handling for craft, pubmatic, and fluct should all be in those modules. see #8596

Copy link
Contributor Author

@eknis eknis Nov 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@patmmccann
I removed the settings for 'craft' that I had added and changed it to only add the imuid parameter.
The previous support will not be removed this time for backward compatibility.
5434fa9

@eknis eknis changed the title ImRtdProvider: add bidder function for craft ImRtdProvider: add imuid param Nov 25, 2024
@eknis eknis requested a review from patmmccann November 26, 2024 03:50
@eknis
Copy link
Contributor Author

eknis commented Dec 6, 2024

@patmmccann
I add to set the value in ortb2 for the adapters to process. Thank you for your review.

@patmmccann patmmccann self-assigned this Dec 26, 2024
@patmmccann patmmccann merged commit 9b15b22 into prebid:master Dec 26, 2024
4 of 5 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.

2 participants