Skip to content
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

Adding support for multi-leg routes in CarPlayManager. #1841

Merged
merged 7 commits into from
Nov 21, 2018

Conversation

JThramer
Copy link
Contributor

Like it says on the box. Fixes #1822.

/cc @mapbox/navigation-ios

@JThramer JThramer added op-ex Refactoring, Tech Debt or any other operational excellence work. CarPlay Bugs, improvements and feature requests on Apple CarPlay labels Nov 14, 2018
@JThramer JThramer added this to the v0.25.0 milestone Nov 14, 2018
@JThramer JThramer self-assigned this Nov 14, 2018
@akitchen akitchen requested a review from 1ec5 November 15, 2018 17:05
MapboxNavigation/CarPlayManager.swift Outdated Show resolved Hide resolved
MapboxNavigation/CarPlayManager.swift Outdated Show resolved Hide resolved
MapboxNavigation/CarPlayManager.swift Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@1ec5 1ec5 merged commit ceb48cc into master Nov 21, 2018
@1ec5 1ec5 deleted the jerrad/carplay-manager-options branch November 21, 2018 21:39
@@ -104,7 +104,7 @@ extension AppDelegate: CPListTemplateDelegate {
// Selected a favorite
if let userInfo = item.userInfo as? [String: Any],
let waypoint = userInfo[CarPlayManager.CarPlayWaypointKey] as? Waypoint {
carPlayManager.calculateRouteAndStart(to: waypoint, completionHandler: completionHandler)
carPlayManager.preview(waypoints: [waypoint], completionHandler: completionHandler)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This method was renamed to previewRoutes(between:completionHandler:). Fixed in cc255c4.

@1ec5 1ec5 added the backwards incompatible changes that break backwards compatibility of public API label Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards incompatible changes that break backwards compatibility of public API CarPlay Bugs, improvements and feature requests on Apple CarPlay op-ex Refactoring, Tech Debt or any other operational excellence work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants