Closed Bug 1746027 Opened 4 years ago Closed 4 years ago

Save to Pocket refresh prefs and nimbus implementation

Categories

(Firefox :: Pocket, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox97 --- fixed

People

(Reporter: thecount, Assigned: thecount)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → sdowne

To test:

You'll need to open and verify each panel, then flip some prefs, and verify each panel again.

To open a panel and verify their query params, do this:

  1. Enable "Disable Popup Auto-Hide" from multiprocess toolbox.
  2. Click the save to Pocket button.
  3. Because "Disable Popup Auto-Hide" is set, the panel should stay open, allowing you to inspect it. You can do this with the "Pick an element from the page" button in the top left of the multiprocess toolbox.
  4. in the inspector, find the url in the browser element, and check the query params.

You want to do steps 1-4 for each panel, signup, saved, and home (click the Pocket button from a newtab to get the home panel).
You need to toggle "Disable Popup Auto-Hide" on and off to open and close panels as needed.

Go about about:config and flip extensions.pocket.refresh.layout.enabled and extensions.pocket.refresh.emailButton.enabled to true
Now test each panel with the prefs flipped to true.

Expected:
Signup panel should have search params: utmCampaign, utmSource, emailButton, layoutRefresh, pockethost, and locale
Home panel should have search params: layoutRefresh, pockethost, and locale
Saved panel should have search params: premiumStatus, fxasignedin, layoutRefresh, pockethost, and locale
emailButton, layoutRefresh should flip based on the prefs. No other search params should be changing from what's on master.

Pushed by sdowne@getpocket.com: https://hg.mozilla.org/integration/autoland/rev/e481b6eb9a48 Save to Pocket refresh prefs and Nimbus implementation r=gvn,andreio
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: