2006785 - Update urlbar-persisted-search-terms-schema.json to not require includeParams

Status: RESOLVED FIXED

(Firefox :: Address Bar, task, P2)

Description

[James Teow [:jteow]]

I'd like to make includeParams an optional property so that we can instead use excludeParams when no inclusion criteria is needed.

Comment 1

[James Teow [:jteow]]

Attachment: Bug 2006785 - Update urlbar-persisted-search-terms-schema.json to not require includeParams - r?daisuke!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/69623bd6652d
https://hg.mozilla.org/integration/autoland/rev/72070de65fb4
Update urlbar-persisted-search-terms-schema.json to not require includeParams - r=daisuke,urlbar-reviewers

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/72070de65fb4

Comment 4

[Phabricator Automation]

firefox-esr140 Uplift Approval Request

• User impact if declined/Reason for urgency: Test-only, fixes a test failure.
• Code covered by automated testing?: yes
• Fix verified in Nightly?: yes
• Needs manual QE testing?: no
• Steps to reproduce for manual QE testing:
• Risk associated with taking this patch: low
• Explanation of risk level: Test-only
• String changes made/needed?: None
• Is Android affected?: no

Comment 5

[Mark Banner (:standard8)]

Attachment: Bug 2006785 - Update urlbar-persisted-search-terms-schema.json to not require includeParams -

Original Revision: https://phabricator.services.mozilla.com/D276940

Comment 6

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-esr140/rev/e77442dc67d5