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.

Sign up for GitHub

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

Jump to bottom

Prebid 8 - Remove second floorMin location at the impression level #9598

Closed
ChrisHuie opened this issue Feb 27, 2023 · 2 comments
Closed

Prebid 8 - Remove second floorMin location at the impression level #9598

ChrisHuie opened this issue Feb 27, 2023 · 2 comments
Labels
8.0 API Change

Comments

@ChrisHuie
Copy link
Collaborator

Issue to track the interest in removing the second floorMin impression level location
@patmmccann
Copy link
Collaborator

patmmccann commented Mar 28, 2023 •
edited
Loading

we support two places for floormin, eg #8396 . We want to support only one, the documented way

@patmmccann patmmccann moved this from Triage to Ready for Dev in Prebid.js Tactical Issues table Apr 24, 2023
@patmmccann
Copy link
Collaborator

patmmccann commented Apr 24, 2023 •
edited
Loading

imp.ext.prebid.data.floormin vs imp.ext.prebid.floormin vs imp.ext.prebid.floors.floorMin; getting rid all but the last as the proper location

@dgirardi dgirardi mentioned this issue May 3, 2023
Merged
1 task
@dgirardi dgirardi moved this from Ready for Dev to PR submitted in Prebid.js Tactical Issues table May 3, 2023
@github-project-automation github-project-automation bot moved this from PR submitted to Done in Prebid.js Tactical Issues table May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
8.0 API Change
Projects
Prebid.js Tactical Issues table
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@patmmccann @ChrisHuie