Directions API. Url provider
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- android, java
- Domaine
- api, build-system
Piste de recherche
Commencez par comparer Router.kt, MapboxDirections.java et l’implémentation interne de RouteUrl.kt, puis examinez les graphes de dépendances des modules présentés dans l’issue. Déterminez comment le comportement partagé des URL d’itinéraire peut être consommé indépendamment sans introduire de dépendances réseau dans Offline Router ; le travail est terminé lorsque les deux types de routeur peuvent utiliser le provider partagé tandis que les interfaces de service-core restent inchangées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
In terms of Navigation for Android exists 2 types of Navigation(Router), it's Offline and Online. Both of them needs Route-URL provider.(Offline and Online are different modules , have own dependency graph and might be used independent).
Online Router's Route URL is provided by MapboxDirections that is a part of services-core library(under the hood is Retrofit(+OkHttp) library)
Problem
Offline Router cannot consume url from services-core because it brings "network" dependencies like Retrofit and OkHttp(they shouldn't be in Offline navigation).
Offline Router has own route-url provider RouteUrl
The main issue here: two classes where the same piece of logic exist.
Solution
- keep
service-corelibrary interfaces as it is - create additional library (like
router-url-provider) that depends onservice-directions-modeland (if java doesn't have clever approach for creating url) additional java library
How dependency tree looks now
Suggestion
Pros
router-url-provider might be used independently on service-core 👍
cc @mapbox/navigation-android @mapbox/maps-android @mapbox/navigation-api
- Langage dominant
- Java
- Étoiles
- 438
- Forks
- 117
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de mapbox/mapbox-java
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
mapbox/mapbox-java#1640 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
mapbox/mapbox-java#1635 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
mapbox/mapbox-java#1619 ·
-
Mapbox-java incompatible with gson 2.13.0 and up - call to internal GSON classes that were renamed Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
mapbox/mapbox-java#1614 · 2 commentaires · 1 réaction ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
mapbox/mapbox-java#1612 · 1 réaction ·
Toutes les issues de mapbox/mapbox-java
Issues similaires
-
awaiting triage bug Causes friction Hop Gui P1 P2 Transforms
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
apache/flink-agents#1152 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
jenkinsci/blueocean-plugin#5417 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
objectionary/eo-graphs#75 ·