Closed Bug 1817038 Opened 2 years ago Closed 2 years ago

Move weather suggestion keywords to Nimbus

Categories

(Firefox :: Address Bar, task, P1)

task

Tracking

()

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

Talked this over with Nan -- Nimbus allows JSON'ed data as variable types, and we should use that to store weather suggestion keywords instead of the quick suggest config in remote settings. It doesn't make a lot of sense to use the RS config because we wouldn't want to change the keywords while experiments are active, and there are no plans right now to enable weather suggestions by default for the whole Suggest population. Using a Nimbus var ties the keywords to specific experiments while still allowing us to modify them right up until the moment any experiment ships.

One drawback is it makes it a little harder for people to manually test the keywords in the absence of an experiment, but that's solvable (by setting up an experiment on stage, running a JS snippet to install a mock experiment, or force enrolling in an actual experiment). We could give the variable a fallback pref and then people could set the pref, but that's extra complication since there's no other reason to use a fallback pref for this.

This reverts the changes from D168757 bug 1814795 related to remote settings. It
moves the weather keywords from the config in remote settings to a new Nimbus
variable called weatherKeywords. Bug 1817038 has more context.

It also adds another variable called weatherKeywordsMinimumLength that lets
the user type prefixes of the keywords instead of the full strings. This is
similar to how sponsored and Wikipedia suggestion keywords contain prefixes,
except here Firefox is computing the prefixes from the full keywords instead of
assuming that the prefixes are in the keyword data.

It also removes the weather.zeroPrefix pref added in D168757. Instead, the
weather suggestion is triggered on zero prefix if a Nimbus experiment isn't
active or if the experiment doesn't define weatherKeywords.

Finally, it makes one unrelated change: The user's search string can now contain
trailing spaces and still match a weather keyword. I don't think it makes sense
for the weather suggestion to be shown when you type "weather" but then it goes
away after you type a space.

Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/75dd0ccb7d55 Move weather suggestion keywords to Nimbus. r=daisuke
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Flags: qe-verify-
Flags: in-testsuite+
Blocks: 1828796
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: