Bug 1847150 Comment 7 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: We will not have the full scope of desired telemetry data on the initial release of the feature, which may make it more difficult to make informed product decisions about the release.
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: Telemetry is considered medium risk based on the following guidelines:
https://wiki.mozilla.org/Release_Management/Uplift_rules

I do not believe this to be a risky change as no new metrics were added. This patch only adds more key/value pairs to existing metrics. 

We are also fairly early in the beta process, which I feel decreases the risk.
* **String changes made/needed**: 
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: 
We will not have the full scope of desired telemetry data on the initial release of the feature, which may make it more difficult to make informed product decisions about the release.
* **Is this code covered by automated tests?**: 
Yes
* **Has the fix been verified in Nightly?**: 
Yes
* **Needs manual test from QE?**: 
No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: 
None
* **Risk to taking this patch**: 
Medium
* **Why is the change risky/not risky? (and alternatives if risky)**: 
Telemetry is considered medium risk based on the following guidelines:
https://wiki.mozilla.org/Release_Management/Uplift_rules
I do not believe this to be a risky change as no new metrics were added. This patch only adds more key/value pairs to existing metrics. 
We are also fairly early in the beta process, which I feel decreases the risk.
* **String changes made/needed**: 
None
* **Is Android affected?**: 
No

Back to Bug 1847150 Comment 7