Skip to content

Implement a new method to change route#3159

Merged
S2Ler merged 1 commit into
mainfrom
s2ler/2948-reroute
Jul 20, 2021
Merged

Implement a new method to change route#3159
S2Ler merged 1 commit into
mainfrom
s2ler/2948-reroute

Conversation

@S2Ler
Copy link
Copy Markdown
Contributor

@S2Ler S2Ler commented Jul 9, 2021

Fixes #2948

Previous method of just updating indexedRoute wasn't robust enough for
cases when the new route has different waypoints than the one it is
replacing. New method allows to update route from the different set of
waypoints.

@S2Ler S2Ler added bug Something isn’t working backwards incompatible changes that break backwards compatibility of public API labels Jul 9, 2021
@S2Ler S2Ler requested review from a team, 1ec5 and chezzdev July 9, 2021 09:20
@S2Ler S2Ler force-pushed the s2ler/2948-reroute branch 5 times, most recently from 10ccf15 to 2adc051 Compare July 9, 2021 09:41
Comment thread CHANGELOG.md Outdated
Comment thread Sources/MapboxCoreNavigation/LegacyRouteController.swift Outdated
@S2Ler S2Ler force-pushed the s2ler/2948-reroute branch from 2adc051 to 7dbaf9a Compare July 19, 2021 15:48
@S2Ler S2Ler added the release blocker Needs to be resolved before the release. label Jul 20, 2021
Copy link
Copy Markdown
Contributor

@Udumft Udumft left a comment

Choose a reason for hiding this comment

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

LGTM

Previous method of just updating `indexedRoute` wasn't robust enough for
cases when the new route has different waypoints than the one it is
replacing. New method allows to update route from the different set of
waypoints.

Fixes #2948
@S2Ler S2Ler force-pushed the s2ler/2948-reroute branch from 7dbaf9a to 113a1dd Compare July 20, 2021 15:08
@S2Ler S2Ler merged commit e308e0c into main Jul 20, 2021
@S2Ler S2Ler deleted the s2ler/2948-reroute branch July 20, 2021 15:44
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 bug Something isn’t working release blocker Needs to be resolved before the release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problems when changing the route in the NavViewController while navigating.

2 participants