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

51Degrees Prebid Server modules documentation #5333

Merged

Conversation

jwrosewell
Copy link
Contributor

Description

51Degrees module enriches an incoming OpenRTB request with 51Degrees Device Data.

51Degrees module sets the following fields of the device object: make, model, os, osv, h, w, ppi, pxratio - interested bidder adapters may use these fields as needed. In addition the module sets device.ext.fiftyonedegrees_deviceId to a permanent device ID which can be rapidly looked up in on premise data exposing over 250 properties including the device age, chip set, codec support, and price, operating system and app/browser versions, age, and embedded features.

🏷 Type of documentation

  • new feature

📋 Checklist

  • Related pull requests in prebid-server and prebid-server-java are linked:

Copy link

netlify bot commented May 22, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 760d976
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/66964552d63c0a0008cedb4e
😎 Deploy Preview https://deploy-preview-5333--prebid-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

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

Overall a good first draft - thanks. A few comments and suggestions

prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
prebid-server/pbs-modules/51degrees-device-detection.md Outdated Show resolved Hide resolved
@justadreamer
Copy link
Contributor

Hi @bretg thank you for providing this thorough review. I've incorporated the changes according to the comments, except for one re: converting the list of parameters into the table - I've checked the preview and indentation seems to reflect the nesting of parameters quite well. Please let us know any further changes needed when you have a chance. Thanks.

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

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

Looks good - thanks @justadreamer

@bretg
Copy link
Contributor

bretg commented Jun 5, 2024

Thanks @justadreamer - unfortunately preview is now broken due to an environment issue. Could you update the Gemfile in this PR to match #5384 ? Then preview would work so I could review the table formatting, which is a common area of problems.

@justadreamer
Copy link
Contributor

Hi @bretg, I've merged the latest master, the preview has been generated, the doc can be previewed here, also added it to the list of modules in the modules index page, so navigable from there too. Thanks.

@bretg
Copy link
Contributor

bretg commented Jun 6, 2024

Nicely done @justadreamer . FWIW, PBS-Java is soon going to support kebab-case-config and snake_case_config as completely the same. So the extra column in the table should be able to go away.

@ChrisHuie ChrisHuie merged commit 39926b8 into prebid:master Jul 18, 2024
5 checks passed
@justadreamer justadreamer deleted the feature/51degrees-device-detection-module branch July 30, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants