Closed Bug 1874949 Opened 9 months ago Closed 8 months ago

Add triggers for Review Checker setting changes

Categories

(Firefox :: Messaging System, task, P1)

task

Tracking

()

RESOLVED WONTFIX
Iteration:
124.1 - Jan 22 - Feb 2

People

(Reporter: aminomancer, Unassigned)

References

(Blocks 1 open bug)

Details

Planned feature callouts 4 and 5 will trigger when a user:

  • disables "automatically open Review Checker"
  • disables Review Checker

This requires 2 new FxMS triggers. There are 2 possible ways to do this:

  1. Listen to the prefs controlled by these toggles and fire the trigger when the prefs changes.
  2. In the forthcoming UI code, in the click listeners' callbacks, directly invoke ASRouter.sendTriggerMessage with an appropriately named trigger.

The latter depends more on the UI, which doesn't exist yet, but it's also more precisely what we want. So the ideal approach depends on when we start work on this. We can start on 1 immediately since we can decide what the names of the prefs will be, and they will be used in the UI code, or else if the UI patch uses different pref names it can also change them in the trigger listener.

Figma spec

Potentially uplift candidate for Fx123, dependency adding review checker settings in about:preferences in progress in Fx124

Iteration: --- → 124.1 - Jan 22 - Feb 2
Points: --- → 5
Priority: -- → P1

preferenceObserver trigger might be good enough for this.

Points: 5 → ---

Not needed thanks to preferenceObserver

Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.