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 dependencies for 2.5.0 release #3834
Conversation
@@ -33,7 +44,6 @@ | |||
* Fixed an issue when `SimulatedLocationManager` could freeze the main thread when working with long routes. The manager now calls delegate methods from a background thread. ([#3672](https://github.com/mapbox/mapbox-navigation-ios/pull/3672)) | |||
* Fixed an issue where initial puck position can be incorrect when `NavigationViewController` is presented. ([#3773](https://github.com/mapbox/mapbox-navigation-ios/pull/3773)) | |||
* Fixed an issue where `UserHaloCourseView` was not correctly shown while changing `CLLocationManager.accuracyAuthorization` and `CLLocationManager.authorizationStatus`. ([#3804](https://github.com/mapbox/mapbox-navigation-ios/pull/3804)) | |||
* Fixed an issue where camera was not centered to the current location during `NavigationMapView` initialization. ([#3826](https://github.com/mapbox/mapbox-navigation-ios/pull/3826)) |
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.
wasn't actually released as part of 2.4.0, since this PR was merged after the release branch was cut
@@ -36,14 +36,14 @@ class Navigator { | |||
.init(setRoutesHandler: { [weak self] route, legIndex, completion in | |||
self?.navigator.setPrimaryRouteForRoute(route, legIndex: legIndex) { [weak self] result in | |||
if result.isValue() { | |||
let routeInfo = result.value as! RouteInfo |
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.
NN now exposes actual generics instead of AnyObject, so no need to cast here and in other places
CHANGELOG.md
Outdated
|
||
### Packaging | ||
|
||
* MapboxNavigation now requires [MapboxMaps v10.5.0-beta.1](https://github.com/mapbox/mapbox-maps-ios/releases/tag/v10.5.0-rc.1). ([#3834](https://github.com/mapbox/mapbox-navigation-ios/pull/3834)) |
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.
* MapboxNavigation now requires [MapboxMaps v10.5.0-beta.1](https://github.com/mapbox/mapbox-maps-ios/releases/tag/v10.5.0-rc.1). ([#3834](https://github.com/mapbox/mapbox-navigation-ios/pull/3834)) | |
* MapboxNavigation now requires [MapboxMaps v10.5.0-rc.1](https://github.com/mapbox/mapbox-maps-ios/releases/tag/v10.5.0-rc.1). ([#3834](https://github.com/mapbox/mapbox-navigation-ios/pull/3834)) |
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.
ah missed that. thanks! will fix now.
Please also rebase this PR onto main to resolve |
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.
Need to rebase this branch, there're some conflicts.
Co-authored-by: Maxim Makhun <maxim.makhun@mapbox.com>
7c7d324
to
63df4eb
Compare
Updating Common to
v21.3.0-rc.2
, Maps to10.5.0-rc.1
and NN to97.0.0