DavidBruant / lotocar

MIT License
1 stars 1 forks source link

La recherche Cahors-Pelacoy ne renvoie aucun résultat et on ne comprend pas ce qu'il se passe #59

Open sabinesafi opened 4 years ago

sabinesafi commented 4 years ago

L'itinéraire demandé ne s'affiche même pas sur la carte, donc peut-être que notre système ne retrouve pas Pelacoy ? C'est perturbant d'avoir la liste des conducteurs et rien qui s'affiche sur la carte ou comme matching adapté. On pourrait indiquer ce qu'il se passe, par exemple "Nous n'arrivons pas à trouver de commune correspondant à la saisie : Pelacoy".

laem commented 4 years ago

Oui c'est bien ça, j'ai l'impression que c'est un lieu-dit de Francoulès et ce n'est pas reconnu comme une destination. L'erreur est claire : "no position for origin or destination (cahors, Pelacoy)"

On pourrait indiquer ce qu'il se passe, par exemple "Nous n'arrivons pas à trouver de commune correspondant à la saisie : Pelacoy".

Ca me semble être une bonne solution, qu'en penses-tu @DavidBruant ?

sabinesafi commented 4 years ago

Pour info Nathalie a ou va ajouter la position GPS de Pelacoy dans le spredsheet dédié donc ça va régler ce cas là. Mais je pense qu'il faut de toute façon afficher un message explicatif quand on n'arrive pas à fournir un itinéraire (le "no position for origin or destination (cahors, Pelacoy)" il est visible en front ? si oui, je ne l'ai pas vu).

laem commented 4 years ago

Il est visible dans la console de développement, qui s'ouvre en appuyant sur F12 en général.

DavidBruant commented 4 years ago

On avait eu le problème précédemment : #40 (ironiquement avec le même lieu-dit :-p )

Je suis d'accord sur la solution (afficher un message dans l'interface). Cette solution va aussi être utile pour l'interface de mise en relation directe

Une question suivante pour la mise en place, c'est la priorisation. Est-ce que c'est important pour ce sprint ou le suivant ? Si ce sprint, est-ce qu'on vire autre chose ? (je n'attends pas de réponse ici, mais je pense qu'une discussion là-dessus serait utile) Pendant la plannif, on est passés à côté de #40. Je rajoute une carte correspondante dans Trello pour visibiliser ce problème