Skip to content

Increment leg index upon way point arrival#1604

Merged
danesfeder merged 1 commit into
masterfrom
dan-leg-auto
Dec 5, 2018
Merged

Increment leg index upon way point arrival#1604
danesfeder merged 1 commit into
masterfrom
dan-leg-auto

Conversation

@danesfeder
Copy link
Copy Markdown
Contributor

Fixes #1487

We currently do not increment the RouteLeg upon arrival of each way point along the route. This essentially means we can only navigate from an origin to a destination with way points in between.

This PR looks at a few criteria before incrementing the index and reseting the NavigationStatus with the new data.

@danesfeder danesfeder merged commit 5d6bdd9 into master Dec 5, 2018
@danesfeder danesfeder deleted the dan-leg-auto branch December 5, 2018 19:14
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1604 into master will increase coverage by 0.09%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master    #1604      +/-   ##
============================================
+ Coverage     22.56%   22.65%   +0.09%     
- Complexity      694      700       +6     
============================================
  Files           189      189              
  Lines          8230     8241      +11     
  Branches        606      609       +3     
============================================
+ Hits           1857     1867      +10     
- Misses         6190     6191       +1     
  Partials        183      183

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Defect to be fixed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants