Confirmed users
95
edits
Data Collection: Difference between revisions
Jump to navigation
Jump to search
m
Clarified that review from *data collection* module owner or peers is required as this caused some confusion
(vladan is no longer a peer) |
Gfritzsche (talk | contribs) m (Clarified that review from *data collection* module owner or peers is required as this caused some confusion) |
||
| Line 35: | Line 35: | ||
* Data collection details must be documented using the in-tree histogram definitions or the in-tree documentation system (.rst files). | * Data collection details must be documented using the in-tree histogram definitions or the in-tree documentation system (.rst files). | ||
* New data collection or changes to data collection must be approved by the module owner or peers by requesting review on the patch which updates the in-tree documentation. | * New data collection or changes to data collection must be approved by the data collection module owner or peers by requesting review on the patch which updates the in-tree documentation. | ||
== Requesting Approval == | == Requesting Approval == | ||
It is our intention to review every new data collection within Firefox, but to do so quickly and with minimal overhead. For every new measurement, even a simple new Telemetry probe, please request approval by setting the feedback flag for the module owner or a peer. Simple requests should be handled within a day. | It is our intention to review every new data collection within Firefox, but to do so quickly and with minimal overhead. For every new measurement, even a simple new Telemetry probe, please request approval by setting the feedback flag for the data collection module owner or a peer. Simple requests should be handled within a day. | ||
More complex requests, and especially requests which add a new kind of data collection mechanism or require changes to the privacy notice, will require more extensive review. Please consider pinging the team about these as they are being designed! Additional discussions/review may include: | More complex requests, and especially requests which add a new kind of data collection mechanism or require changes to the privacy notice, will require more extensive review. Please consider pinging the team about these as they are being designed! Additional discussions/review may include: | ||