-
Notifications
You must be signed in to change notification settings - Fork 117
Conversation
The URL format is the one shown in the text message screenshot? I should be able to make this work on Android as well. |
@barbeau here's the URL format I'm using:
with query params:
|
Thanks! What's the significance of the |
They map to different actions. Try opening https://www.onebusaway.co/regions/1/stops/1_11420/arrivals?trip_id=1_30938281&service_date=1478242800000&stop_sequence=4 vs https://www.onebusaway.co/regions/1/stops/1_11420 on your desktop. |
b26574b
to
8c98922
Compare
This feature wasn't so much driven by specific user request as it is driven to replace the hacky workarounds I've seen users use to accomplish the same thing. Here are the key points of this commit: 1. Allows users to share the trip they are taking with another person via text message or any other form of communication vended through UIActivityViewController. 2. Trips are shared via URL. These URLs encode the ID of the region that their trips belong to, which means that they are globally unique. They also do not depend on individual OBA region servers, which means that the iOS app doesn't need region server information hardcoded into it. 3. People with OneBusAway for iOS are taken to the trip itself when they tap on one of these links. 4. People without OBA for iOS are taken first to http://onebusaway.co, and then redirected to the appropriate region server's web page for that trip. ------- Fixes: * Fixes #833 - 'Follow My Trip' Feature * Fixes #840 - Implement Deep Linking * Fixes #847 - Add option to delete individual recent stops ------- Deep Links/Shared Trips: * Deep link router class * Create OBATripDeepLink model object * Shared Trip infrastructure for model DAO and the user defaults-based persistence layer * Display shared trips on the Recent tab * Delete shared trips older than 24 hours * Make it possible to render the ArrivalAndDeparture controller from a Trip Deep Link object * Start overhauling OBANavigationTarget * Wire up navigation target stuff for OBARecentStopsViewController * Shoehorn in a way to launch differently in response to deep links. * Unify navigation target, deep link, and 3D Touch quick action navigation systems Miscellaneous: * Move -escapePathVariable into OBAURLHelpers * Fix credits - turns out that I didn't end up adding the credits.html file back to the project when I did the big project reorg last month. Oops. Cell Swiping: * Add third party library YMSwipeTableViewCell to allow us to display icons on swiped cells (only on the stop view controller for now!) * Improved swipe menu for departure rows on the stops controller Icons: * New bookmark icons * Massively improve tab bar icons * Update launch storyboard - remove text and ensure that the icons are the correct, new ones * Add full sized icons from Noun Project to the Resources folder
8c98922
to
899d29f
Compare
Wow - I'm horrified to admit I didn't even know that existed. Were the |
No reason to be horrified, I registered the domain last week :)
I needed something I could throw a simple Rails app on for this sort of purpose. Happy to transition it to onebusaway.org at a later date, but I figured that would be a lengthier process than slapping up something quick. |
Ok, that makes me feel better :). Any objection to using this for other OBA purposes? We were thinking about a centralized URL scheme related to the Microsoft Research Embedded Social features, and it would be great to leverage what you've already created. |
No, not at all. I was hoping we could, actually! |
That's great! Thanks for implementing this, I think it will be very useful. |
After looking at this a little more closely - IMHO the current Same with https://www.onebusaway.co/regions/1/stops/1_11420/arrivals?trip_id=1_30938281&service_date=1478242800000&stop_sequence=4, on the web it takes you to the trip.action site. So it seems a more natural name for this endpoint would be However, I currently know nothing of how OBA.co is implemented. @aaronbrethorst Thoughts on this? I'm happy to stick with the current structure if it causes a lot of work, just my two cents looking at this for the first time. |
A renaming would be trivial. Happy to do it. |
@barbeau I just deployed a change that changes |
Thanks! Sure, I can create new repo - what do you want the name to be? |
I'm not particularly emotionally attached to any particular name. |
This feature wasn't so much driven by specific user request as it is driven to replace the hacky workarounds I've seen users use to accomplish the same thing.
Here are the key points of this commit:
Fixes:
Deep Links/Shared Trips:
Miscellaneous:
Cell Swiping:
Icons:
Walkthrough: