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

Exposes the MapboxMap in NavigationView with a getter method. #642

Merged
merged 5 commits into from
Jan 17, 2018

Conversation

devotaaabel
Copy link
Contributor

The iOS version exposes the MapboxMap to developers, so this PR is for the consideration of similarly adding a getter to the Android version.

Copy link
Contributor

@danesfeder danesfeder left a comment

Choose a reason for hiding this comment

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

Looks good, before you merge, let's add a note in the javadoc that this will return a null MapboxMap until the view is initialized.

@devotaaabel devotaaabel merged commit e2baf00 into master Jan 17, 2018
@devotaaabel devotaaabel deleted the devota-give-mapboxmap-access branch January 17, 2018 22:23
@danesfeder danesfeder mentioned this pull request Jan 23, 2018
14 tasks
TranThuong pushed a commit to TranThuong/mapbox-navigation-android that referenced this pull request Jan 26, 2018
* master:
  Update Strings with Transifex Translations (mapbox#657)
  Added embedded navigation example (mapbox#654)
  Release 0.9.0 (mapbox#663)
  Update README.md for 0.9.0 (mapbox#664)
  Add Maneuver type exit rotary constant (mapbox#653)
  Update Maps and Services dependencies  (mapbox#661)
  Add FasterRouteDetector to check for quicker routes while navigating  (mapbox#638)
  Directions API Banner instructions (mapbox#582)
  NavigationTelemetry update cue for changing configurations (mapbox#648)
  Moved WaypointNavigationActivity from the SDK to the test app (mapbox#652)
  Exposes the MapboxMap in NavigationView with a getter method (mapbox#642)
  Add language to NavigationViewOptions with default from RouteOptions (mapbox#635)

# Conflicts:
#	libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/navigation/NavigationEngine.java
#	libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/navigation/NavigationService.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants