Skip to content

Nativo: Updating JS documentation#6016

Merged
muuki88 merged 30 commits into
prebid:masterfrom
life360-oss:master
May 15, 2025
Merged

Nativo: Updating JS documentation#6016
muuki88 merged 30 commits into
prebid:masterfrom
life360-oss:master

Conversation

@rafataveira
Copy link
Copy Markdown
Contributor

@rafataveira rafataveira commented Apr 29, 2025

This pull request updates the documentation for the Nativo bidder in dev-docs/bidders/nativo.md, reverting back Prebid JS documentation to its original after the latest updates.

Changes to field definitions:

  • Replaced ntv_url with url and marked it as optional, updating the description and example to align with the Nativo platform's current usage.
  • Replaced ntv_ptd with placementId, marking it as required, and updated its description and example to reflect the new naming convention and data type.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 29, 2025

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 6819b5a
🔍 Latest deploy log https://app.netlify.com/projects/prebid-docs-preview/deploys/68250a988c756b000822c36d
😎 Deploy Preview https://deploy-preview-6016--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 project configuration.

@muuki88
Copy link
Copy Markdown
Contributor

muuki88 commented May 2, 2025

Thanks for the detailed explanation 🙏

Are the old parameters still in use by publishers? Is there some migration necessary?
Was this falsely merged into the prebid docs or are there some prebid versions that actually use those docs? And is there a related PR?

What I'm getting at - a "in those prebid versions you must use the other format" disclaimer feels necessary, if any of the questions above are answered with yes 😅

@rafataveira
Copy link
Copy Markdown
Contributor Author

Thanks for the detailed explanation 🙏

Are the old parameters still in use by publishers? Is there some migration necessary? Was this falsely merged into the prebid docs or are there some prebid versions that actually use those docs? And is there a related PR?

What I'm getting at - a "in those prebid versions you must use the other format" disclaimer feels necessary, if any of the questions above are answered with yes 😅

@muuki88 Your concern is valid; however, there are no modifications to the adapters themselves. This update ensures consistency with the existing solution.

@rafataveira
Copy link
Copy Markdown
Contributor Author

@muuki88 Just following up.
There is no need for changes. This documentation update was requested by our product team to be more consistent with our existing solution.

@muuki88 muuki88 merged commit c481c79 into prebid:master May 15, 2025
6 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.

4 participants