Closed Bug 1205928 Opened 4 years ago Closed 4 years ago

Auto-disconnect when showing the migration infobar

Categories

(Firefox :: Sync, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 44
Iteration:
44.2 - Oct 19
Tracking Status
firefox44 --- fixed

People

(Reporter: markh, Assigned: markh)

References

Details

Attachments

(3 files)

It's possible that people still using legacy Sync are getting the migration infobar, but for whatever reason don't want to upgrade and don't realize that clicking the "upgrade" button will take then to Sync prefs where they can "unlink this device" (which is the terminology used by legacy sync IIRC). Eg, see bug 1205437 comment 5.

IOW, there may be users who are simply putting up with the migration bar because they aren't sure how to make it go away without causing (perceived) pain.

Given Sync is very soon EOL, we should consider updating the migration UI to offer a button saying "disconnect" or "stop using Sync" or something similar.

Alternatively: given 43 is the earliest we could realistically get this out to users (with 44 being more likely), we could decide to silently and automatically disconnect - by then Sync will have not been working for a significant amount of time so it seems fairly clear they don't care.

Whatchya think?
Flags: firefox-backlog+
> Given Sync is very soon EOL, we should consider updating the migration UI to offer a button
> saying "disconnect" or "stop using Sync" or something similar.

If it's a relatively simple change, then yeah, I think this would be worthwhile.  There's clearly a lot of users syncing and seeing the notice and not doing anything about it.
Given that the earliest we can get out changes is 43 or 44, is it worth doing anything?
Flags: needinfo?(markh)
(In reply to Chris Karlof [:ckarlof] from comment #2)
> Given that the earliest we can get out changes is 43 or 44, is it worth
> doing anything?

Anecdotally, I was helping my parents the other day and saw the 2 stacked bars, which they had no idea what to do with, nor how to make go away. I suspect that they would still have had the bars in version 45 had I not helped them remove it.

So I think we should do something - but given these time-frames, I'm thinking we should just silently and automatically disconnect (if the migration offer still exists then, I think it is safe to say these people just want it gone). Chris, what do you think?
Flags: needinfo?(markh) → needinfo?(ckarlof)
A string-less fix sounds reasonable. What are the conditions under which we would "silently and automatically disconnect"?
Flags: needinfo?(ckarlof) → needinfo?(markh)
(In reply to Chris Karlof [:ckarlof] from comment #4)
> A string-less fix sounds reasonable. What are the conditions under which we
> would "silently and automatically disconnect"?

I was thinking aggressively - as soon as we got an EOL notification from the EOLinator (ie, at the time we currently show the infobar). The next question is whether we should also remove the migration code - if we think it may be useful in the future to enterprises, then I'd say we add a new pref (services.sync.disconnectOnEOL) that we set to true, but enterprises using Sync 1.1 may prefer to set to false so they can get the same migration experience. Alternatively, we just ignore enterprise and make the migration their local problem.
Flags: needinfo?(markh)
Priority: -- → P1
We agreed that in 44, we will:
* disconnect from Sync as soon as we get an EOL message (probably based on a pref in comment 5)
* change the text of the infobar to say "You have been disconnected [learn more][connect to sync]"
* profit!!??
Summary: Add a "disconnect" button to the migration infobar? → Auto-disconnect when showing the migration infobar?
I believe we also agreed to not bother with the preference mentioned in comment 5 and just delete all relevant migration code from the tree. This reduces the engineering overhead of the migration code at the cost of leaving enterprise users of Sync on their own when it comes to migration from Sync 1.1 to Sync 1.5 - but it seems unlikely our migration could would actually be of benefit in that environment.
Assignee: nobody → markh
Summary: Auto-disconnect when showing the migration infobar? → Auto-disconnect when showing the migration infobar
CC Nick so he is aware of our plans. tl;dr - this bug is likely to land in 44 (Jan 2016) or 45 (March 2016) by which time I think we can safely assume that anyone still seeing this bar simply doesn't know how to get rid of it, so we will automatically disconnect them.
Lots more code removal :) This patch automatically disconnects from legacy Sync once it sees an EOL notification.
Attachment #8673482 - Flags: review?(adw)
Attached image auto-disconnect.png
Ryan, are you happy with this UX? Note that once the infobar goes away for any reason it never comes back - the user is just in the "not connected to Sync" state.
Attachment #8673483 - Flags: ui-review?(rfeeley)
(oh - and the "learn more" link takes the user to the same place it did when we actually migrated, which isn't perfect but IMO it is good enough)
Attachment #8673482 - Flags: review?(adw) → review+
Iteration: --- → 44.2 - Oct 19
Attached file Sync Broken.pdf
I suppose just getting rid of the bars makes sense at this unfortunate juncture, but I'm confused. The UX spec had covered the "Sync broken" state  (attached)very clearly. The design was last updated December 19, 2014. What happened?
Flags: needinfo?(wmaggs)
Flags: needinfo?(edwong)
:rfeeley - that hard disconnect task just got lost, it needed a discrete bug or brought up on the sync meetings. Markh informally brought up this idea and took the initiative to create a patch.
Flags: needinfo?(edwong)
Yeah, we always planned on doing the hard EOL UX "later", but then the Sync decommissioning approached quickly and we ran out of time to get that done and on the trains before the servers were turned off - the end of May was the deadline for that and we missed it.
Flags: needinfo?(wmaggs)
sorry Mark, had to back this out for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=5234673&repo=fx-team
Flags: needinfo?(markh)
Flags: needinfo?(markh)
https://hg.mozilla.org/mozilla-central/rev/ddb5b8e191c9
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Attachment #8673483 - Flags: ui-review?(rfeeley) → ui-review+
Depends on: 1227205
You need to log in before you can comment on or make changes to this bug.