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

Sovrn bid adapter add ortb2 device #11784

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jwrosewell
Copy link
Contributor

Type of change

  • Feature
  • Updated bidder adapter

Description of change

This PR enhances the Sovrn bidder request by incorporating device data from the global ORTB2 object. Existing values in the Sovrn device object will be overridden if they are also present in the global ORTB2 object. However, values unique to Sovrn or not found in the global ORTB2 object will remain unchanged.

Motivation

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 Sovrn’s users to benefit from device object improvements.

Other information

cc: @TWrightSovrn

Copy link

Whoa there, partner! 🌵🤠 We wrangled some duplicated code in your PR:

  • modules/deepintentBidAdapter.js has 9 duplicated lines with modules/sovrnBidAdapter.js
  • modules/deepintentBidAdapter.js has 14 duplicated lines with modules/sovrnBidAdapter.js
  • modules/deepintentBidAdapter.js has 12 duplicated lines with modules/sovrnBidAdapter.js

Reducing code duplication by importing common functions from a library not only makes our code cleaner but also easier to maintain. Please move the common code from both files into a library and import it in each. Keep up the great work! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants