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

Rerouting customiztion rollback #3668

Merged
merged 1 commit into from
Jan 12, 2022
Merged

Conversation

Udumft
Copy link
Contributor

@Udumft Udumft commented Dec 23, 2021

This PR rolls back configuration for dangerous maneuver during rerouting and custom route providing mechanism. This is required due to upcoming NavNative RerouteController integration which will be most likely incompatible with current implementation. In order to not publish API which may get unsupported in the next version, we do a rollback now.
Reverted features will be available in future versions (most likely 2.3.0).

@Udumft Udumft added release blocker Needs to be resolved before the release. Core Work related to core navigation and integrations. v2.x Related to version 2.x. labels Dec 23, 2021
@Udumft Udumft added this to the v2.2-rc milestone Dec 23, 2021
@Udumft Udumft requested a review from a team December 23, 2021 13:35
@Udumft Udumft self-assigned this Dec 23, 2021
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

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

We’ll need to mention the removal of these features in the next prerelease’s release notes.

### Location tracking

* Added the `RouterDelegate.router(_:initialManeuverBufferWhenReroutingFrom:)` method and related `NavigationServiceDelegate` and `NavigationViewControllerDelegate` methods to configure rerouting buffering. ([#3454](https://github.com/mapbox/mapbox-navigation-ios/pull/3454))
* Added `RouterDelegate.router(_:requestBehaviorForReroutingWith:)` and related `NavigationServiceDelegate` and `NavigationViewControllerDelegate` methods to provide customization for rerouting mechanism by providing a user-defined route as a reroute. ([#3472](https://github.com/mapbox/mapbox-navigation-ios/pull/3472))
Copy link
Contributor

Choose a reason for hiding this comment

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

😞 Looking forward to a revival of this useful feature!

…rous maneuver during rerouting and custom route providing mechanism. CHANGELOG entries removed too
@Udumft Udumft force-pushed the vk/rollback-reroute-customization branch from fde0895 to 3f32db0 Compare January 12, 2022 09:04
@mapbox-github-ci-issues-public-1

No breaking changes detected in MapboxCoreNavigation

@mapbox-github-ci-issues-public-1

No breaking changes detected in MapboxNavigation

@Udumft Udumft merged commit 2e2401e into main Jan 12, 2022
@Udumft Udumft deleted the vk/rollback-reroute-customization branch January 12, 2022 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Work related to core navigation and integrations. release blocker Needs to be resolved before the release. v2.x Related to version 2.x.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants