New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Maps v10.10.0-rc.1, NavNative 122.0.0, Common v23.2.0-rc.3 #4283
Conversation
@@ -209,7 +209,7 @@ public class MapboxRoutingProvider: RoutingProvider { | |||
let directionsUri = settings.directions.url(forCalculating: options).removingSKU().absoluteString | |||
var requestId: RequestId! | |||
|
|||
requestId = router.getRouteForDirectionsUri(directionsUri) { [weak self] (result, origin) in | |||
requestId = router.getRouteForDirectionsUri(directionsUri, options: GetRouteOptions(timeoutSeconds: nil)) { [weak self] (result, origin) in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we set a default timeout for the route request? @mapbox/navigation-ios
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we have 60 seconds timeout in Directions, but since this setting allows a 'default' nil - we should probably use that.
…lternative routes requesting to avoid testReroutingFromALocationSendsEvents crash
Description
This Pr is to update the dependency to Maps v10.10.0-rc.1, NavNative v122.0.0, Common v23.2.0-rc.3
Implementation
Screenshots or Gifs