Skip to content

PubxAi and Pubmatic RTD: remove import of continueAuction #14230

@patmmccann

Description

@patmmccann

"It was brought to my attention that this pattern has a precedent:

/**
* This RTD module has a dependency on the priceFloors module.
* We utilize the createFloorsDataForAuction function from the priceFloors module to incorporate price floors data into the current auction.
*/
import { createFloorsDataForAuction } from './priceFloors.js'; // eslint-disable-line prebid/validate-imports

Same issue applies there; it makes the module larger and more brittle. However it can (and hopefully does) work, as long as the floors logic doesn't change.

It may be worth updating the floors module to support this use case - allow RTD providers to set floors data.

In the meanwhile, my recommendation remains to remove the dependency on the internals of floors, and update this to just set config for it; but since we have a precedent I'm OK with moving this forward as it is."

Originally posted by @dgirardi in #12732 (comment)

also in #14104 there was additional discussion. It seems Pubmatic team has a fix planned

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

Status

Ready for Dev

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions