More dynamic floormin

[patmmccann]

Type of issue

Feature

Description

#5805 allowed for minimums to floors as a page level variable, but we need floorMin at slot, and potentially other granularity, etc level.

[patmmccann]

Prebid.js/modules/priceFloors.js

Line 359 in 061eb3c

resolvedFloorsData.data = getFloorDataFromAdUnits(adUnits);

seems to allow for the object with the floor min

Prebid.js/modules/priceFloors.js

Line 373 in 061eb3c

if (resolvedFloorsData.hasOwnProperty('floorMin')) resolvedFloorsData.data.floorMin = resolvedFloorsData.floorMin;

to come from the ad unit; is this accurate?

If so, could this be solved with documentation? We need floormin for each ad unit, not necessarily arbitrary syntax.

[patmmccann]

I like @bretg proposal: Support a floormin override on adunit.ortb2Imp.ext.prebid.floorMin ; marking this as ready for dev

[robertrmartinez]

So the change is to:

Update the price floors module to look for floorMin at adunit.ortb2Imp.ext.prebid.floorMin first, and then do whatever it does now to find it if not?

@bretg @patmmccann

[patmmccann]

yes; this would allow for it to vary over ad units, no?

[robertrmartinez]

Yep it is a good idea!

I'll try and get this and #8339 done over the weekend or something!

[muuki88]

This looks super useful. Does this also work if there's no floor information at all?

Currently we set our floors something like this

floors: {
  currency: config.currency.adServerCurrency,
  schema: {
    delimiter: '|',
    fields: ['mediaType']
  },
  values: {
    // provided from somewhere else
    '*': priceRule.floorprice
  }
}

With this extension would it be possible to leaf the floors property undefined and only set

ortb2Imp: {
  ext: {
    prebid: {
      floorMin: priceRules.floorprice
    }
  }
}

[patmmccann]

I got so excited i merged without docs, apologies :)

we're implementing shortly.

[patmmccann]

@muuki88 did you figure out the answer to that in test?

[muuki88]

@patmmccann unforunately not. I'll put this on my teams agenda

[patmmccann]

I haven't tested, but I suspect that would not work. You might be able to get away with just the currency or something though.