Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

AnnotationManager.java refactor #6067

Closed
tobrun opened this issue Aug 18, 2016 · 0 comments · Fixed by #6913
Closed

AnnotationManager.java refactor #6067

tobrun opened this issue Aug 18, 2016 · 0 comments · Fixed by #6913
Assignees
@tobrun
Labels
Android Mapbox Maps SDK for Android refactor
Milestone
android-v5.0.0

Comments

@tobrun
Copy link
Member

tobrun commented Aug 18, 2016

The idea behind this refactor is to extract logic from MapboxMap and use it as facade to route the public api internally to the concept of Annotation Manager. This mimics the behaviour we have in core. We should back up this with unit tests for adding/removing and managing selection states.
@tobrun tobrun added refactor Android Mapbox Maps SDK for Android labels Aug 18, 2016
@tobrun tobrun self-assigned this Nov 4, 2016
@tobrun tobrun added this to the android-v5.0.0 milestone Nov 4, 2016
@tobrun tobrun mentioned this issue Nov 4, 2016
Merged
4 tasks
@tobrun tobrun mentioned this issue Nov 25, 2016
Closed
@zugaldia zugaldia mentioned this issue Feb 14, 2017
Closed
10 tasks
@zugaldia zugaldia mentioned this issue Feb 28, 2017
Closed
9 tasks
@zugaldia zugaldia mentioned this issue Mar 10, 2017
Closed
8 tasks
@tobrun tobrun mentioned this issue Mar 17, 2017
Closed
10 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tobrun tobrun

Labels
Android Mapbox Maps SDK for Android refactor
Projects
None yet
Milestone
android-v5.0.0
Development

Successfully merging a pull request may close this issue.

6067 annotation manager mapbox/mapbox-gl-native
1 participant
@tobrun