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

Gumgum Bid Adapter: changes for parameter consistency #8742

Merged
merged 13 commits into from
Jul 26, 2022

Conversation

john-ivan
Copy link
Contributor

Type of change

  • Refactoring (no functional changes, no api changes)

Description of change

Updated the parameter for consistency with our server side implementations

@patmmccann
Copy link
Collaborator

Appears to be a breaking change, why not support both

@patmmccann patmmccann self-requested a review July 26, 2022 20:19
@patmmccann patmmccann self-assigned this Jul 26, 2022
@patmmccann patmmccann merged commit 92a5051 into prebid:master Jul 26, 2022
JacobKlein26 pushed a commit to nextmillenniummedia/Prebid.js that referenced this pull request Feb 9, 2023
* Gumgum - ADTS-175 Support multiple GG params

* ADJS-1165-prebid-adaptor-changes-to-support-jp-products

* made tweaks to the skin product for the gumgumBidAdapter

* added test for new product id

* updated skins parameter for consistency

* updated test for updated skins parameter

Co-authored-by: Lisa Benmore <[email protected]>
Co-authored-by: John Bauzon <[email protected]>
jorgeluisrocha pushed a commit to jwplayer/Prebid.js that referenced this pull request May 23, 2023
* Gumgum - ADTS-175 Support multiple GG params

* ADJS-1165-prebid-adaptor-changes-to-support-jp-products

* made tweaks to the skin product for the gumgumBidAdapter

* added test for new product id

* updated skins parameter for consistency

* updated test for updated skins parameter

Co-authored-by: Lisa Benmore <[email protected]>
Co-authored-by: John Bauzon <[email protected]>
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