-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Documentation for 152media RTD module #6135
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for prebid-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
layout: page_v2 | ||
title: 152media RTD Module | ||
display_name: 152media | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
page_type: module | ||
module_type: rtd | ||
module_code : oftmediaRtdProvider | ||
enable_download : true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is enable_download: true
meant to signify to publishers that this module employs third-party script injection as a part of its operation?
And if true, does this meet the definition of "prominent disclosure"?
A Real-Time Data module may load external code if it requires publisher registration and there’s a prominent disclosure on the module documentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No no 😁 It means that it can be downloaded from the download page. True is a solid value!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@muuki88 - thanks!
Do you know if, as this documentation currently stands, there is enough "disclosure" that the RTD module downloads third-party scripts as a part of its work? It looks like pubs have to register with 152media to work with them and their module, so maybe it becomes obvious at that point. Curious what you think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right 👍 Thanks for pointing this out. Added a suggestion below
This module is maintained by [152media](mailto:[email protected]) and requires the global `rtdModule` to function. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This module is maintained by [152media](mailto:[email protected]) and requires the global `rtdModule` to function. | |
This module is maintained by [152media](mailto:[email protected]) and requires the global `rtdModule` to function. | |
{% include dev-docs/loads-external-javascript.md %} | |
Thanks for the callout @jefftmahoney . There should indeed be a disclaimer that this module loads 3rd party js that is not open source or vetted by prebid.
🏷 Type of documentation
📋 Checklist