mapbox / mapbox/mapbox-navigation-android
Merge guidance view sub-type components into a single one
- Dominant language
- Kotlin
- Stars
- 651
- Forks
- 321
- PR merge metrics
- No merged PRs in 30d
Description
Refs https://github.com/mapbox/mapbox-navigation-android/pull/6239#pullrequestreview-1091711679.
This ticket tracks deprecating the `MapboxSignboardApi/View`, `MapboxJunctionApi/View`, and `MapboxSapaApi/View` (if introduced) classes, in favor of a `MapboxImageGuidanceApi` and `MapboxImageGuidanceView` (or something along these lines) that would expose distinct methods to download any of the sub-types and reuse the rest of the setup.
cc @abhishek1508 @kmadsen @tomaszrybakiewicz @cafesilencio
Contributor guide
Research direction
Start by reading the referenced pull request review and locating MapboxSignboardApi/View, MapboxJunctionApi/View, and MapboxSapaApi/View in the repository. Determine the intended unified MapboxImageGuidanceApi/View design and how each guidance sub-type should be exposed. Done means the deprecated sub-type components are replaced or deprecated in favor of the agreed single component without losing their setup behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- kotlin
- Domain
- mobile
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100