Closed Bug 1362291 Opened 8 years ago Closed 8 years ago

Build URL Router for Deep Links

Categories

(Firefox for iOS :: General, enhancement)

Other
iOS
enhancement
Not set
normal

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.
Assignee: nobody → bmunar
Blocks: 1355636
Whiteboard: [mma]
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.