Skip to content

[Android Auto] Add MapboxCarMapLoader#6530

Merged
kmadsen merged 1 commit intomainfrom
NAVAND-841-add-MapboxCarMapLoader
Nov 3, 2022
Merged

[Android Auto] Add MapboxCarMapLoader#6530
kmadsen merged 1 commit intomainfrom
NAVAND-841-add-MapboxCarMapLoader

Conversation

@kmadsen
Copy link
Copy Markdown
Contributor

@kmadsen kmadsen commented Nov 1, 2022

Description

Moving this class to the sdk because it is essentially copy pasted in all our solutions. This also removes a step for setting up the sdk.

Similar to the CompassWidget, it is optional to use so it is not part of the MapboxCarContext. But i'm considering making this enabled by default, and then you disable it in order to provide your own style loader. That way the behavior can be supported through MapboxCarContext and MapboxCarOptions

@kmadsen kmadsen added the Android Auto Bugs, improvements and feature requests on Android Auto. label Nov 1, 2022
@kmadsen kmadsen requested a review from a team as a code owner November 1, 2022 01:11
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 1, 2022

Codecov Report

Merging #6530 (10a4c84) into main (fe31488) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #6530   +/-   ##
=========================================
  Coverage     70.47%   70.47%           
  Complexity     5033     5033           
=========================================
  Files           729      729           
  Lines         28404    28404           
  Branches       3383     3383           
=========================================
  Hits          20019    20019           
  Misses         7113     7113           
  Partials       1272     1272           

@kmadsen kmadsen force-pushed the NAVAND-841-add-MapboxCarMapLoader branch 4 times, most recently from 4e19151 to d0b391d Compare November 1, 2022 20:09
@kmadsen kmadsen force-pushed the NAVAND-841-add-MapboxCarMapLoader branch 2 times, most recently from a1b4f66 to e97d5dc Compare November 2, 2022 17:34
@kmadsen kmadsen force-pushed the NAVAND-841-add-MapboxCarMapLoader branch 2 times, most recently from ad9713f to 7ec141f Compare November 3, 2022 17:43
@kmadsen kmadsen force-pushed the NAVAND-841-add-MapboxCarMapLoader branch from 7ec141f to 10a4c84 Compare November 3, 2022 18:17
@kmadsen kmadsen merged commit 53f8c17 into main Nov 3, 2022
@kmadsen kmadsen deleted the NAVAND-841-add-MapboxCarMapLoader branch November 3, 2022 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Android Auto Bugs, improvements and feature requests on Android Auto.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants