Closed
Bug 1362291
Opened 8 years ago
Closed 8 years ago
Build URL Router for Deep Links
Categories
(Firefox for iOS :: General, enhancement)
Tracking
()
RESOLVED
FIXED
People
(Reporter: sdaswani, Assigned: bmunar, Mentored)
References
Details
(Whiteboard: [mma])
James proposed the following for URL routing in Fennec iOS: https://docs.google.com/document/d/1UqK_V3JCiMxWGbcyP1nskEzlg-EycvYsXuddcn2uvws/edit#
Please build a URL Router for Deep Links with the following specs:
* The URL Router should be consulted after the current deep link support in Fennec's App Delegate, i.e., maintain that deep linking logic and then if the deep link request is not satisfied by that logic, use the new URL Router.
* The URL router should consume a deep link and do the following:
** If the App is backgrounded or foregrounded, the URL Router should immediately (modally?) show the requested UI. When the UI is dismissed, the user should be returned to whatever UI they were previously on.
** If the App is not active, the URL Router should immediately (modally?) show the requested UI. When the UI is dismissed, the user should be placed into whatever UI the App shows upon launch.
In both cases there should be no weird pushing of intermediate UIs - there deep links should lead directly where they need to be and when dismissed end up where they should be directly.
Comment 1•8 years ago
|
||
I think we should probably avoid building one from scratch, but using an existing one.
e.g. https://github.com/skyline75489/SwiftRouter
(In reply to James Hugman [:jhugman] [@jhugman] from comment #1)
> I think we should probably avoid building one from scratch, but using an
> existing one.
>
> e.g. https://github.com/skyline75489/SwiftRouter
OK thanks, Bryan will check this out. I agree we shoudn't reinvent the wheel.
Another spec:
* If a user clicks a deep link and the App is not installed, it should open the App Store page for Firefox.
Will be included as part of this PR: https://bugzilla.mozilla.org/show_bug.cgi?id=1355636
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•