638834 - Provide more descriptive error message on JPAKE timeout

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, enhancement)

Description

[Wesley Johnston (:wesj)]

When Sync hits any error during JPAKE we show the message: "Firefox could not connect to Sync. Would you like to try again?" This will show if you bring up the sync screen and then just let it timeout (or if you are very slow entering your key on desktop). It would also show if you entered your key on desktop, but for some reason sync could not update your phone.

Maybe we can provide a message that was a bit more descriptive, but applied in both situations. "Sync codes can not be verified" or something...

Comment 1

[Thomas Arend [:tarend]]

Marking 2.0next. We should show (a) a specific error message, and (b) a call to action for the user that describes how to resolve the issue or where to find more information.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Do we have support for this in Sync? Is this something we can do in Fennec?

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Can we make some progress on this?

Comment 4

[Wesley Johnston (:wesj)]

Attachment: Patch

Sorry. Been avoiding this because I'm not sure what the right answer is. On device we can detect a timeout and show a different message. This makes us write:
"Your sync codes could not be verified. Would you like to try again?"

as opposed to our normal cheery:

"Fennec could not connect to Sync. Would you like to try again?"

Try again and Manual Setup seem like the only two actions we would want to direct most users to (and Manual setup is kinda debateable). We could replace it with a link to SUMO. I will put up a second patch for that.

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Patch 2 - Help button

Help button patch. Replaces "Manual Setup" button with "Get Help". Directs the user to https://support.mozilla.com/kb/Firefox%20Sync%20is%20not%20working

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 540087 [details] [diff] [review]
Patch 2 - Help button

Code looks fine, but lets get a UI r+ before landing

Comment 7

[Wesley Johnston (:wesj)]

Attachment: Screenshot

Comment 8

[Madhava Enros [:madhava]]

This is specifically a JPAKE error, and I don't want to start explaining JPAKE in these messages. We can probably do better here, but replacing manual setup isn't it, I think.

Comment 9

[Mark Finkle (:mfinkle) (use needinfo?)]

Let's just close this until we really have a good plan or need