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

bump NN to 161.1.0 #7613

Merged
merged 3 commits into from
Nov 13, 2023
Merged

bump NN to 161.1.0 #7613

merged 3 commits into from
Nov 13, 2023

Conversation

dzinad
Copy link
Contributor

@dzinad dzinad commented Nov 13, 2023

Description

Screenshots or Gifs

Copy link

github-actions bot commented Nov 13, 2023

Changelog

Features

Bug fixes and improvements

  • Supported ETC 2.0 payment method. [#dd](https://github.com/mapbox/mapbox-navigation-android/pull/dd)

Known issues ⚠️

Other changes

Android Auto Changelog

Features

Bug fixes and improvements

  • The app is now considered as the one in active navigation only when an active route is set to MapboxNavigation. Previously it was always considered active. [#7366](https://github.com/mapbox/mapbox-navigation-android/pull/7366)
  • When Android Auto host tells the app to stop active navigation because another app starts navigating, now SDK will enter FreeDrive mode instead of stopping trip session completely. [#7366](https://github.com/mapbox/mapbox-navigation-android/pull/7366)

Copy link

codecov bot commented Nov 13, 2023

Codecov Report

Merging #7613 (37df3a6) into release-v2.17 (224e804) will not change coverage.
Report is 3 commits behind head on release-v2.17.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##             release-v2.17    #7613   +/-   ##
================================================
  Coverage            74.11%   74.11%           
  Complexity            6164     6164           
================================================
  Files                  834      834           
  Lines                33157    33157           
  Branches              3961     3961           
================================================
  Hits                 24574    24574           
  Misses                7050     7050           
  Partials              1533     1533           

@dzinad dzinad marked this pull request as ready for review November 13, 2023 14:00
@dzinad dzinad requested a review from a team as a code owner November 13, 2023 14:00
@@ -13,15 +13,15 @@ ext {
// version which we should use in this build
def mapboxNavigatorVersion = System.getenv("FORCE_MAPBOX_NAVIGATION_NATIVE_VERSION")
if (mapboxNavigatorVersion == null || mapboxNavigatorVersion == '') {
mapboxNavigatorVersion = '161.0.1'
mapboxNavigatorVersion = '161.1.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there anything to add to the changelog?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, they supported ETC 2.0, I added that to changelog and supported it on mapbox-java side as well.

@VysotskiVadim VysotskiVadim added the needs backporting Requires cherry-picking to a currently running release branch label Nov 13, 2023
@dzinad dzinad mentioned this pull request Nov 13, 2023
@dzinad dzinad enabled auto-merge (rebase) November 13, 2023 16:28
@dzinad dzinad merged commit b956ff2 into release-v2.17 Nov 13, 2023
2 of 4 checks passed
@dzinad dzinad deleted the dd-nn-161.1.0 branch November 13, 2023 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs backporting Requires cherry-picking to a currently running release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants