Closed Bug 1938392 Opened 1 year ago Closed 1 year ago

Add integratedSidebar pref to Nimbus

Categories

(Firefox Graveyard :: Shopping, task, P2)

Tracking

(firefox138 verified)

VERIFIED FIXED
138 Branch
Tracking Status
firefox138 --- verified

People

(Reporter: fchasen, Assigned: fchasen)

References

Details

(Whiteboard: [fidefe-RCSidebar])

Attachments

(1 external file)

We'll need to be able to control which sidebar is being used with Nimbus, so we will want to add browser.shopping.experience2023.integratedSidebar to the FeatureManifest.yaml

Blocks: 1946223
No longer blocks: 1916005
Assignee: nobody → fchasen
Status: NEW → ASSIGNED

Adds the integratedSidebar pref to the shopping2023 Nimbus feature and separates the enabling prefs for the migrated Review Checker in the main sidebar and the Shopping sidebar.

  • Updates browser.shopping.experience2023.integratedSidebar to be the only enabling pref for the migrated Review Checker sidebar.
  • Prevents browser.shopping.experience2023.enabled from disabling the migrate sidebar and uses it as the pref for the ShoppingSidebar actors.
  • Removes the browser.shopping.experience2023.shoppingSidebar pref as it is no longer needed.
  • Updates any tests the used these prefs to the new use.
  • Init's the two sidebar differently in ShoppingUtils based on the Nimbus prefs.
Blocks: 1950570
Pushed by fchasen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c6f7ddb9661f Add integratedSidebar pref to Nimbus. r=shopping-reviewers,firefox-desktop-core-reviewers ,sidebar-reviewers,mossop,Gijs,rking
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch

Verified as fixed in our latest Nightly 138.0a1 (2025-03-27) The browser.shopping.experience2023.shoppingSidebar pref is no longer displayed in our Nightly build

Status: RESOLVED → VERIFIED
Blocks: 1953542
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: