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

How to implement Native in Adapter #10493

Closed
QuentinGallard opened this issue Sep 15, 2023 · 2 comments
Closed

How to implement Native in Adapter #10493

QuentinGallard opened this issue Sep 15, 2023 · 2 comments
Assignees

Comments

@QuentinGallard
Copy link
Contributor

Type of issue

Question

Description

Hi, I'm from Smilewanted and we have several questions about adding the Native Support to our Adapter.

The documentation here https://docs.prebid.org/prebid/native-implementation.html suggests to pass a Native OpenRTB 1.2 bidrequest in mediatypes.native.ortb, but every other adapters use the legacy version https://docs.prebid.org/prebid/native-implementation-legacy.html or convert the ortb request in legacy via the convertOrtbRequestToProprietaryNative function in native.js.

Is there any reason not to use the ortb method rather than the legacy method?

Is the documentation for the ortb method too recent?

What is the recommended way to implement Native in Adapters ?

@dgirardi
Copy link
Collaborator

The goal is to eventually move everything to ORTB, so that's what we recommend for new implementations. There are a number of adapters that do native ORTB, but all of them get it "for free" through the ortbConverter library. For example:

supportedMediaTypes: [BANNER, VIDEO, NATIVE],

Here's the logic - but as you can see there's not much in it, since it's just passing ORTB objects back and forth:

https://github.com/prebid/Prebid.js/blob/master/libraries/ortbConverter/processors/native.js

From what I can tell Smilewanted does not talk ORTB - so you may need to do more legwork in the adapter. See also this discussion about the gaps between legacy and ORTB:

#10249

@dgirardi dgirardi moved this from Ready for Dev to Needs OP in Prebid.js Tactical Issues table Sep 25, 2023
@patmmccann
Copy link
Collaborator

closing as stale

@patmmccann patmmccann closed this as not planned Won't fix, can't repro, duplicate, stale Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants