dfp-prebid-setup icon indicating copy to clipboard operation
dfp-prebid-setup copied to clipboard

Support run-of-network line items (no targeted placements)

Open ptomasroos opened this issue 7 years ago • 4 comments

dfp.exceptions.BadSettingException: The setting "DFP_TARGETED_PLACEMENT_NAMES" must contain at least one DFP placement ID.

IMHO this should default to Run of network in the case of empty array not yield a error. On our case I see exactly no reason to have this targeted against specific placements, or adunits the reason is that one needs to generate many more than just having 1 per bidder which runs for all adunits.

ptomasroos avatar Apr 24 '17 11:04 ptomasroos

This makes sense.

When we add this feature, I'd like to protect people who blindly run the script without configuring settings properly, because I believe the most common use case is to target to placements. We could safeguard by adding DFP_ALLOW_NO_INVENTORY_TARGETING that defaults to false.

@ptomasroos Interested in contributing this?

kmjennison avatar Apr 24 '17 22:04 kmjennison

Thanks for fixen the issues btw @kmjennison, I hacked it locally, will clean up and send a PR in a week or two

ptomasroos avatar May 05 '17 07:05 ptomasroos

@ptomasroos do you have that PR ready or a sample of what you did? We need run of network as well, not specific adunits or placements. This could save me many hours of tedious work :)

kevinlisota avatar Jul 30 '17 03:07 kevinlisota

I'd be interested in Run of Network also, does anyone managed to get it done? Thanks!

pboisso avatar Sep 23 '17 00:09 pboisso