Closed
Bug 1126584
Opened 9 years ago
Closed 9 years ago
Support signing in to Firefox Account via embedding accounts.firefox.com
Categories
(Firefox for iOS :: Firefox Accounts, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: nalexander, Assigned: nalexander)
References
Details
Attachments
(1 file)
This ticket tracks embedding accounts.firefox.com into the iOS app and implementing (enough of) the Sync postMessage protocol to pass Firefox Account credentials back to Swift.
Assignee | ||
Comment 1•9 years ago
|
||
Hey peeps, here's a first experiment at embedding the FxA sign in flow into Firefox for iOS. I glued it in to an existing button in settings, just for testing. The patch shows an interstitial loading/error page which is replaced with the background loaded webview. If you log in with a valid FxA (pointing at dev servers right now, you may need to make a new account) you'll see the settings UI update the email address. There's no persistence yet.
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
Attachment #8555573 -
Flags: review?(sarentz)
Attachment #8555573 -
Flags: review?(bnicholson)
Comment 2•9 years ago
|
||
Comment on attachment 8555573 [details] [review] Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/117 I added a bunch of notes to the PR. Some of it is just Swift code style stuff. But there are also some bigger questons about how things talk to eachother. I +'d this patch because I don't want to hold this up but I do think we should at least chat about some of the bigger points. They can also be followup refactoring bugs.
Attachment #8555573 -
Flags: review?(sarentz) → review+
Comment 3•9 years ago
|
||
Comment on attachment 8555573 [details] [review] Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/117 Sorry for the late review. Looks pretty good to me.
Attachment #8555573 -
Flags: review?(bnicholson) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://github.com/mozilla/firefox-ios/commit/c7b48fa9dde51c8be25f814cc631418f88435f11
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•