-
Notifications
You must be signed in to change notification settings - Fork 34
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 electronic horizon example to new API #104
Conversation
979395c
to
a63a8f1
Compare
navigationMapView.mapView.update { | ||
$0.location.puckType = .puck2D() | ||
} | ||
navigationMapView.mapView.on(.styleLoaded) { [weak self] _ in | ||
self?.setupMostProbablePathStyle() | ||
} | ||
|
||
// TODO: Provide a reliable way of setting camera to current coordinate. | ||
DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { |
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.
In the latest NavSDK version camera API was changed and this code doesn't compile anymore.
This code will be rewritten using new API here: #101
Shall we separate changes related to SwiftLint and Electronic Horizon? (e.g. move changes which are needed for SwiftLint to another PR)? I think camera related changes already landed to |
@azarovalex mind reviving this PR? |
After updating this PR with the latest changes from the main it looks like the main branch already has all the changes, so I'll close the PR. |
Electronic horizon public API was changed here mapbox/mapbox-navigation-ios#2906.
This PR changes the "Upcoming intersections" example to use the new API.