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

DFP : vconp & vpa managing playback methods ? #12562

Open
GuillQuirin opened this issue Dec 10, 2024 · 3 comments
Open

DFP : vconp & vpa managing playback methods ? #12562

GuillQuirin opened this issue Dec 10, 2024 · 3 comments

Comments

@GuillQuirin
Copy link

Hello

Google recommends us to set the "vconp" & "vpa" param in the adrequest in order to improve the video monetization, I can see in the code they depends on the playback method.

However I'm wondering how to declare the playback method when the player initiates as usual, and then, keep playing its playlist (ie : value 1 + 7).

Is it impossible to declare these 2 parameters together ?

Thank you

@patmmccann
Copy link
Collaborator

If you set 1,7 as an array for mediaTypes.video.playbackmethod and we set vconp based on presense on 7 in the array, does that solve?
image

@patmmccann patmmccann moved this from Triage to Ready for Dev in Prebid.js Tactical Issues table Dec 10, 2024
@GuillQuirin
Copy link
Author

@patmmccann It seems the vconp param in the Dfp adapter file doesn't allow any values other than 7.
Replacing .every() by .some() could fix it indeed.

image

@dgirardi
Copy link
Collaborator

As a workaround, You can set those parameters directly using the params option of buildVideoUrl. For example,

pbjs.adServers.dfp.buildVideoUrl({
    // ...
    params: {
        vconp: '2',
        vpa: 'auto'
    }
});

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

No branches or pull requests

3 participants