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 : add ORTB2 device data to request payload #12008

Merged
merged 4 commits into from
Oct 11, 2024

Conversation

jwrosewell
Copy link
Contributor

Type of change

  • Feature
  • Updated bidder adapter

Description of change

This PR enhances the GumGum bid request by incorporating device data from the global ORTB2 object.

The device object has previously been populated by simplistic parsers, if at all, and was inaccurate as a result. Prebid now benefits from RTD modules such as 51Degrees that enrich all the device object fields including Apple iPhone model category and device ID. The PR enables GumGum’s users to benefit from device object improvements.

ORTB2 values have been mapped on to gumgum bid request parameter schema as follows:

  • ua: ORTB2.device.ua
  • dnt: ORTB2.device.dnt
  • os: ORTB2.device.os
  • osv: ORTB2.device.osv
  • dt: ORTB2.device.devicetype
  • lang: ORTB2.device.language
  • make: ORTB2.device.make
  • model: ORTB2.device.model
  • ppi: ORTB2.device.ppi
  • pxratio: ORTB2.device.pxratio
  • foddid: (FiftyOne)51Degrees device ID

Other information

cc: @MartinGumGum

@jwrosewell jwrosewell marked this pull request as ready for review September 27, 2024 08:38
@jwrosewell
Copy link
Contributor Author

@MartinGumGum - this PR will ensure GumGum benefits from publishers efficiently populating the device object with enriched data including iPhone model categories and other features not obtainable via SUA and UA. As publishers progress rollout this becomes more critical.

const _device = deepAccess(ortb2Data, 'device') || {};

// set device data params from ortb2
const _deviceRequestParams = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@BohdanVV can you add ip and ipv6 to this?

Copy link
Contributor

Choose a reason for hiding this comment

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

@BohdanVV can you add ip and ipv6 to this?

Done 👌🏻

Copy link
Contributor

@MartinGumGum MartinGumGum left a comment

Choose a reason for hiding this comment

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

LGTM

@MartinGumGum
Copy link
Contributor

@BohdanVV Circleci build has failed could you please rerun and merge?

@BohdanVV
Copy link
Contributor

BohdanVV commented Oct 9, 2024

Hi @MartinGumGum, I don't have the necessary permissions to do that, but I'll make sure to ask someone who does. :)

@MartinGumGum
Copy link
Contributor

Hi @patmmccann , maybe you can help us rerun circlei? We don't have permission.

@ChrisHuie ChrisHuie self-assigned this Oct 10, 2024
@ChrisHuie ChrisHuie changed the title GumGum Bid Adapter: Add ORTB2 device data to request payload GumGum Bid Adapter : add ORTB2 device data to request payload Oct 10, 2024
@ChrisHuie ChrisHuie merged commit 953a7ac into prebid:master Oct 11, 2024
6 checks passed
pdamoc pushed a commit to missena-corp/Prebid.js that referenced this pull request Oct 22, 2024
…#12008)

* GumGum Bid Adapter: Add ORTB2 device data to request payload

* GumGum Bid Adapter: add `ip` and `ipv6` to request payload

---------

Co-authored-by: Bohdan V <[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.

5 participants