Bug 1833520 Comment 4 Edit History

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

Original comment by
on 
### Beta/Release Uplift Approval Request
* **User impact if declined**: Users might not see the expected experiment experience
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: See Test Plan https://phabricator.services.mozilla.com/D178256
* **List of other uplifts needed**: none
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Doesn't change default behavior and might only be used in 114 experiment that would go through experiment QA before launching
* **String changes made/needed**: none
* **Is Android affected?**: No
Revision 1 by
on 
### Beta/Release Uplift Approval Request
* **User impact if declined**: Users might not see the expected experiment experience
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: See Test Plan https://phabricator.services.mozilla.com/D178256
* **List of other uplifts needed**: none
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Doesn't change default behavior and might only be used in 114 experiment that would go through experiment QA before launching
* **String changes made/needed**: none
* **Is Android affected?**: No
Revision 2 by
on 
### Beta/Release Uplift Approval Request
* **User impact if declined**: Users might not see the expected experiment experience
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes comment 6
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: See Test Plan https://phabricator.services.mozilla.com/D178256
* **List of other uplifts needed**: none
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Doesn't change default behavior and might only be used in 114 experiment that would go through experiment QA before launching
* **String changes made/needed**: none
* **Is Android affected?**: No

Back to Bug 1833520 Comment 4