-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
feature: adding and tracking refresh
depth param
#12083
Comments
You linked the develop branch, also what's wrong with Prebid.js/libraries/riseUtils/index.js Line 126 in bfea83e
|
Fixed the link thanks. Would it make sense for the ORTB2 converter to pick this up at least from |
Hmmm, the refresh proposal is rather publisher unfriendly. I think we'd have a bit of a revolt if we included it by default. Also some of these settings seem hard to infer? |
sounds like a good topic to bring to the PMC. |
Discussion result: #11184 (comment) may be "the sense of the publishers" on this. what's the goal on providing this? It also doesnt seem possible to detect the publisher future intent to resfresh or not and fill out the refsettings object. |
thanks for the link @patmmccann. I agree that automatically sending half the data (just refresh depth which can be inferred) is a bit half baked. I'll close this for now. Thx |
Type of issue
Feature request
Description
ORTB has a standard
refresh
parameter that defines the refresh depth (https://github.com/InteractiveAdvertisingBureau/openrtb2.x/blob/main/2.6.md#objectrefresh). Would prebid.js be able to keep track and increment this based on the calls torequestBids
? Seems like something that would be useful vs having the pub try and configure it properly with each call.The text was updated successfully, but these errors were encountered: