Closed Bug 1790321 Opened 2 years ago Closed 2 years ago

In a profile with sync configured from before featureTour was enabled, the `browser.firefoxView.featureTour` pref stays at its default value indefinitely

Categories

(Firefox :: Messaging System, defect)

defect

Tracking

()

VERIFIED FIXED
106 Branch
Tracking Status
firefox106 --- verified

People

(Reporter: dholbert, Assigned: mviar)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This is related to bug 1790319, but I figured it'd be worth spinning off separately.

In my regular browsing profile (where I have Firefox Sync configured and have other devices syncing as well), I didn't get any particular tour-invitation when running a Nightly with the Firefox View feature-tour enabled. (I checked on two different machines; no tour on either machine in my regular profile, though I do get a tour in a fresh profile.)

I'm not sure if this lack-of-a-tour-for-existing-sync-users was intentional or not; but it seems to mean that the tour pref is just left at its default value indefinitely (since taking the tour is what changes the pref value).

And this persistent/stuck "tour not yet seen" pref-status impacts the layout of the page (perhaps unintentionally) as described in bug 1790319; and I suspect it might have other side effects as well?

Anyway, just filing this bug to be sure it's something that's known/understood. If we'd like for existing users with Sync configured to see the Firefox View tour and to get this pref eventually set to its TRUE-flavored value, we need to take some sort of action to make that happen, because it's not happening right now.

Whiteboard: [fidefe-2022-mr1-firefox-view]

Venetia -- Wanted to ping you on this one as well to bring some clarity on the question around the Tour.

Flags: needinfo?(vtay)
Component: Firefox View → Messaging System
Whiteboard: [fidefe-2022-mr1-firefox-view]

Existing sync users should see Firefox View feature tour the first time. @mviar FYI

Flags: needinfo?(vtay) → needinfo?(mviar)
Assignee: nobody → mviar
Flags: needinfo?(mviar)

I verified that I can reproduce this ("lack of a tour") in a fresh profile, FWIW.

Starting with a fresh Firefox profile, using tofay

  1. Visit about:preferences#sync and sign in to an existing Firefox account.
  2. Restart Firefox
  3. Visit your Firefox View page (for the first time in this profile).

ACTUAL RESULTS:
No tour.

EXPECTED RESULTS:
Should see a tour, per comment 2.

Thanks, Daniel. Working on a patch for this now, looks like we just need some tweaks to the message configurations.

Thanks!

(In reply to Venetia from comment #2)

Existing sync users should see Firefox View feature tour the first time. @mviar FYI

Side note: is the intent for folks to get a tour on every device that they have Firefox Sync configured on? Or do we just want them to see a tour on one device and then not on any other devices?

If that is the intent (tour on all devices): good, that seems likely what will happen after this bug here is fixed.
If that's not the intent: then perhaps we need to ensure that the pref value itself gets sync'ed, or we need to take some other action to avoid repeating the tour everywhere.

Attachment #9294335 - Attachment description: Bug 1790321 - In a profile with sync configured from before featureTour was enabled, the pref stays at its default value indefinitely → Bug 1790321 - Ensure that the Firefox View feature tour is displayed when users already have tab sync enabled

(In reply to Daniel Holbert [:dholbert] from comment #5)

If that's not the intent: then perhaps we need to ensure that the pref value itself gets sync'ed, or we need to take some other action to avoid repeating the tour everywhere.

Syncing across devices addressed in bug 1790669

Pushed by mviar@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/883aa9e5438c Ensure that the Firefox View feature tour is displayed when users already have tab sync enabled r=jprickett
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch

Hmm, this bug's commit is be included in today's Nightly, but I'm still not seeing any sort of tour when I view Firefox View (and I still have browser.firefox-view.feature-tour set to its default value {"message":"FIREFOX_VIEW_FEATURE_TOUR","screen":"FEATURE_CALLOUT_1","complete":false}).

So this bug still seems to be present for me, in the sense that this pref is still stuck at its default value I'm still not being offered a Firefox View tour.

Do we know why that might happen? Should I file a new bug as a clone of this bug vs. should I reopen this bug?

Flags: needinfo?(mviar)

Aha, I restarted Firefox just-in-case (didn't think it would matter because I didn't have an update pending), and now I did get a tour on my first visit to Firefox View. :) So this does seem to be fixed after all, though there was just a delay (or a need for an additional restart) in order for the tour to get triggered.

Flags: needinfo?(mviar)

I have verified this issue using the latest Firefox Beta 106.0b3 (Build ID: 20220922185808) on Windows 10x64, macOS 12.3.1 and LinuxMint 20.2 x64 using the steps provided in Comment 3 and I can confirm the following:

  • The Firefox View feature Tour is successfully displayed after logging into an existing Firefox account.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: