Conversation
Several places are incomplete and are marked with @todo
It's tempting to use a valid EID but if anyone were to play with it then we could get the device id it contains mixed up with others. With this commit, I've gone to a placeholder value that is syntatically valid/representative but will not decrypt.
Add Parrable ID System Docs
bretg
left a comment
There was a problem hiding this comment.
There are a number of ID sub-modules pending -- there will almost certainly be collisions to fix by the time this is merged, but it looks ok other than needing to know how you want to appear in the OpenRTB. Search for 'eids' in the doc.
|
@bretg and @jdwieland8282, Thank you for the feedback. With respect to openRTB are you referring to bidders with prebid server adapters. The snippet above appears to be from the Prebid server section of the Docs. On a related note, we are not clear how to modify or inform prebid server to include our cookies in the openRTB object. We can certainly extend the docs, but how does prebid server know to pickup the storage that we have configured in prebid.js? |
|
Hey @justinegreene sorry yes I was referring to prebid server adapters, from a documentation standpoint this docs PR should describe how PB server adapters should be prepared to read your id. wrt, your question about modifying prebid server prebid/Prebid.js#4178 (comment) |
|
@bretg @jdwieland8282 all reviews addressed in this PR. please consider merging. thanks. |
jdwieland8282
left a comment
There was a problem hiding this comment.
thanks for adding, looks good.
|
resolved conflicts after merging liveintent. merging this now |
Adds documentation for the forthcoming Parrable ID system submodule.