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

Yieldmo requiring deprecated media param 'placement' #10915

Closed
pieisgood opened this issue Jan 8, 2024 · 4 comments
Closed

Yieldmo requiring deprecated media param 'placement' #10915

pieisgood opened this issue Jan 8, 2024 · 4 comments
Assignees

Comments

@pieisgood
Copy link

const placement = validate('video.placement', val => isDefined(val), paramRequired);

This should be updated to support the new plcmt field and not have placement as a requirement since it is now deprecated.

@patmmccann
Copy link
Collaborator

@fowler446 we'd love for the video committee to weigh in on if requiring deprecated parameters should be disallowed in prebid 9

@fowler446
Copy link
Collaborator

We should absolutely drop this field from being required. Doesn't make sense to require a deprecated field.

@fowler446
Copy link
Collaborator

@desidiver any thoughts here? I don't think we should be requiring a deprecated field.

@fowler446
Copy link
Collaborator

Closing this issue in favour of this global list of needed placement changes
#10452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

3 participants