(Hidden by Administrator)
Bug 1879714 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.
(Hidden by Administrator)
### Beta/Release Uplift Approval Request * **User impact if declined**: We regularly update `search-telemetry-v2.json` via Remote Settings. If we update entries in that file with component `type`'s not present in the schema, `browser/components/search/test/unit/test_search_telemetry_config_validation.js` will fail. Part of this change will loosen the restriction from a list of allowable strings to a general regular expression to avoid having to do these future uplifts. * **Is this code covered by automated tests?**: Yes * **Has the fix been verified in Nightly?**: No * **Needs manual test from QE?**: No * **If yes, steps to reproduce**: * **List of other uplifts needed**: bug 1878818 * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Changing the schema should only affect `browser/components/search/test/unit/test_search_telemetry_config_validation.js`. * **String changes made/needed**: * **Is Android affected?**: Yes ### ESR Uplift Approval Request * **If this is not a sec:{high,crit} bug, please state case for ESR consideration**: We regularly update `search-telemetry-v2.json` via Remote Settings. If we update entries in that file with component `type`'s not present in the schema, `browser/components/search/test/unit/test_search_telemetry_config_validation.js` will fail. Part of this change will loosen the restriction from a list of allowable strings to a general regular expression to avoid having to do these future uplifts. * **User impact if declined**: * **Fix Landed on Version**: 125 * **List of other uplifts needed**: bug 1878818 * **Risk to taking this patch**: Low * **Why is the change risky/not risky? (and alternatives if risky)**: Changing the schema should only affect `browser/components/search/test/unit/test_search_telemetry_config_validation.js`.