1193462 - Tapping Sign in on Empty Sync panel should take you to the Sign in flow, not Settings first

Status: RESOLVED FIXED

(Firefox for iOS :: Home screen, defect)

Description

[Robin Andersen [:tecgirl]]

As I understand it, since the flow is a webview is must be loaded this way and cannot be loaded from the Sync panel. 

If possible, the user should be taken straight to the Sign in flow within Settings, not left at Settings.

Is there a way to do push them directly to the Sign in flow? They may see a short hop but they would end up in the right place and not have to tap it on their own.

Comment 1

[Richard Newman [:rnewman]]

I don't know of a good reason why we couldn't just take them straight into sign-in; we just haven't built it yet.

Comment 2

[Aaron Train [:aaronmt]]

Again doesn't this and bug 1187318 depend on bug 1161151?

Comment 3

[Richard Newman [:rnewman]]

This is also true for the tour.

Comment 4

[Stephan Leroux [:sleroux]]

Attachment: https://github.com/mozilla/firefox-ios/pull/974

Comment 5

[Richard Newman [:rnewman]]

I'm testing this now, particularly the case where you already have an account.

Comment 6

[Richard Newman [:rnewman]]

This is a no go. As it stands this PR takes you to a new Sign In page from the tour when you already have an account, and that will cause havoc.

At the very least it should take you to the manage page -- the one you get to when you tap your account name in Settings.

Alternatively we should consider taking signed-in users to Settings, where they'll be able to see the state of their account, tap Sync Now, or just head back to the browser.

Other than that, this looks good.

Comment 7

[Brian Nicholson (:bnicholson)]

Comment on attachment 8651960 [details] [review]
https://github.com/mozilla/firefox-ios/pull/974

Sounds like rnewman is more familiar with this flow than I am, so passing review to him.

Comment 8

[Stephan Leroux [:sleroux]]

Comment on attachment 8651960 [details] [review]
https://github.com/mozilla/firefox-ios/pull/974

I've updated the patch to include redirecting the user to the settings page from the signin button on the tour if the user is already signed in. Also added PR comment fixes:

https://github.com/mozilla/firefox-ios/commit/a772ee986ed7d7d14461d1db5b046e3dcededdb3

Comment 9

[Stephan Leroux [:sleroux]]

Merged