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

AppNexus Bid Adapter: Add support for 'video' in custom Native fields #12345

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

JaroVDH
Copy link
Contributor

@JaroVDH JaroVDH commented Oct 21, 2024

Type of change

  • Updated bidder adapter

Description of change

Copy 'video' over to custom native fields so it can be used in native creatives directly.
It already exists in the base native object, but that's not accessible as asset as it's not a default field in Prebid.

I've also kept the 'video' field in the base object for backwards compatibility, in case people are reading it from the bid directly already.

Other information

This PR will also be forwarded to the Xandr teams for their approval.

@patmmccann
Copy link
Collaborator

#12272

@patmmccann
Copy link
Collaborator

related #10321

@jsnellbaker
Copy link
Collaborator

@JaroVDH do you have a test page with a currently working example of the native video to validate against these changes?

@JaroVDH
Copy link
Contributor Author

JaroVDH commented Oct 22, 2024

@jsnellbaker Do you mean for the backwards compatible video in the bid response?
If so, I created a quick test page that lists if video can still be found in the bid itself, and also already lists if the video is received as asset (the change in this PR):
https://advertising-cdn.dpgmedia.cloud/test-pages/appnexus-native-video.html

@jsnellbaker jsnellbaker merged commit 6ae5759 into prebid:master Oct 25, 2024
6 checks passed
@JaroVDH JaroVDH deleted the appnexusbidadapter-video branch October 25, 2024 17:43
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