If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Add Account State "MigratedFromSync11"

RESOLVED FIXED in mozilla37

Status

Cloud Services
Firefox Sync: Cross-client
RESOLVED FIXED
3 years ago
10 months ago

People

(Reporter: nalexander, Assigned: nalexander)

Tracking

unspecified
mozilla37
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
In Android Sync, as soon as we see a sentinel (see Bug 1017433), we'll remove the existing Old Sync account as create new Firefox Account.  That Firefox Account will be in a "Migrated from Sync 11" state until the user interacts with it: namely, they need to enter their password.

This new state will:

1) display notifications that the user needs to do something to complete the migration on this device:

https://people.mozilla.org/~nalexander/screenshots/Sync.Migration.Join.the.Party.Notifications.png

2) tell the user that the migration isn't completed in the status activity:

https://people.mozilla.org/~nalexander/screenshots/Sync.Migration.Status.Activity.png

3) provide some UI helping the user complete the migration when they tap on an appropriate place:

https://people.mozilla.org/~nalexander/screenshots/Sync.Migration.Sign.In.png

4) show some UI in the most visible part of Fennec saying that the upgrade needs user input to continue:

https://people.mozilla.org/~nalexander/screenshots/Sync.Migration.Remote.Tabs.Panel.png

5) possibly show some UI saying that the upgrade has completed and sync has started, offering something like a Back to Browsing button.

https://people.mozilla.org/~nalexander/screenshots/Sync.Migration.Completed.png
(Assignee)

Comment 1

3 years ago
rfeeley: the first 4 are "join the party" messaging, the last is "you joined the party!" messaging.  Technically this state is identical to "password required", so I've included the existing messaging that a user would see when they need to update their credentials.  You own:

1) the text and subtext of the notification, and where it goes (to the status activity or to the "complete migration" activity);

2) the icon, color, and text of the box in the status activity;

3) the entire visual design of the "join the party" form;

4) the text (but not the colors) in the "Remote Tabs" home panel;

5) the entire visual design of the "you joined the party" screen (including whether there should be such a screen).

I'm aware that this is not very similar to the carefully curated flows you've been circulating on Desktop, but c'est la vie :)  I will try to crib copy from your flows; if you are happy with the existing colors, things will be pretty simple.  If not, you'll need to work with antlam to finalize visuals.
Flags: needinfo?(rfeeley)
(Assignee)

Comment 2

3 years ago
(In reply to Nick Alexander :nalexander from comment #1)
> rfeeley: the first 4 are "join the party" messaging, the last is "you joined
> the party!" messaging.  Technically this state is identical to "password
> required", so I've included the existing messaging that a user would see
> when they need to update their credentials.  You own:
> 
> 1) the text and subtext of the notification, and where it goes (to the
> status activity or to the "complete migration" activity);

15:30 rfeeley_: nalexander h1. Ready to finish Sync migration?
15:30 rfeeley_: h2. Tap to connect user@domain.com

Or:

Finish upgrading Sync?
Tap to connect user@domain.com

Or:

You have upgraded Sync!
Tap to resume syncing. {start,resume,initiate,commence}
(Assignee)

Comment 3

3 years ago
(In reply to Nick Alexander :nalexander from comment #1)
> rfeeley: the first 4 are "join the party" messaging, the last is "you joined
> the party!" messaging.  Technically this state is identical to "password
> required", so I've included the existing messaging that a user would see
> when they need to update their credentials.  You own:
> 
> 1) the text and subtext of the notification, and where it goes (to the
> status activity or to the "complete migration" activity);
> 
> 2) the icon, color, and text of the box in the status activity;

rfeeley suggests the same as on Desktop:

Sign in to connect to your new Firefox Account.

Or:

Tap to connect to your new Firefox Account.
(Assignee)

Comment 4

3 years ago
> 3) the entire visual design of the "join the party" form;

rfeeley suggest following the "Sign in to continue" screen from desktop, like:

h1. Sign in to continue
h2. Sign in to {complete,finish} upgrading Sync.

or:

h1. Sign in to continue
h2. Upgrading Sync can transfer a lot of data.  It's best to be on a WiFi network.

rfeeley feels that adding a "Forget this account" or "Don't upgrade this device" is a foot-gun and that deleting the account with the Android Settings and/or our FxA status activity is good enough.
(Assignee)

Comment 5

3 years ago
Created attachment 8528646 [details] [review]
Link to Github pull-request: https://github.com/mozilla-services/android-sync/pull/509

Here are the android-sync parts of the UI.  Words and lyrics basically as rfeeley and I discussed on Vidyo (and summarized on the ticket).
Flags: needinfo?(rfeeley)
Attachment #8528646 - Flags: review?(rnewman)
(Assignee)

Comment 6

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/705b1be87804
(Assignee)

Comment 7

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/d71b55679c07
(Assignee)

Comment 8

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/ae7ca57ce760
https://hg.mozilla.org/integration/fx-team/rev/3397a68199cc
(Assignee)

Comment 9

3 years ago
Pushed; ni to me to screencap for rfeeley.
Flags: needinfo?(rfeeley)
(Assignee)

Updated

3 years ago
Flags: needinfo?(rfeeley) → needinfo?(nalexander)
https://hg.mozilla.org/mozilla-central/rev/ae7ca57ce760
https://hg.mozilla.org/mozilla-central/rev/3397a68199cc
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Nit: Wi-Fi should not be spelled WiFi.
https://hg.mozilla.org/mozilla-central/diff/ae7ca57ce760/mobile/android/base/locales/en-US/sync_strings.dtd#l1.20
Attachment #8528646 - Flags: review?(rnewman) → review+
(Assignee)

Comment 12

3 years ago
(In reply to Nick Alexander :nalexander from comment #9)
> Pushed; ni to me to screencap for rfeeley.

Okay, here we go:

1) display notifications that the user needs to do something to complete the migration on this device:

https://people.mozilla.org/~nalexander/screenshots/Migration.Notification.png

2) tell the user that the migration isn't completed in the status activity:

https://people.mozilla.org/~nalexander/screenshots/Migration.Status.Activity.png

3) provide some UI helping the user complete the migration when they tap on an appropriate place:

https://people.mozilla.org/~nalexander/screenshots/Migration.Sign.In.png

4) show some UI saying that the upgrade has completed and sync has started, offering something like a Back to Browsing button.

https://people.mozilla.org/~nalexander/screenshots/Migration.Completed.png
Flags: needinfo?(rfeeley)
(Assignee)

Updated

3 years ago
Flags: needinfo?(nalexander)
Depends on: 1128092

Updated

3 years ago
Flags: needinfo?(rfeeley)
Comment hidden (spam)
Comment hidden (spam)
You need to log in before you can comment on or make changes to this bug.