GeotrekCE / Geotrek-mobile

Cross-platform native app
https://geotrek.fr
BSD 2-Clause "Simplified" License
24 stars 10 forks source link

Treks list - Order by distance #206

Closed camillemonchicourt closed 5 years ago

camillemonchicourt commented 5 years ago

Dans le V2, si le terminal était localisé, alors les randos étaient ordonnées par distance de leur point de départ part rapport à la position GPS du terminal. Sinon par ordre alphabétique, ce qui ne semble pas le cas non plus ?

https://github.com/GeotrekCE/Geotrek-mobile/issues/23

bastyen commented 5 years ago

562a310

camillemonchicourt commented 5 years ago

Tests sur Android 9, quand je clique sur Trier par Distance, rien ne se passe, la liste reste ordonnée par Ordre alphabétique.

Je suis bien localisé car quand j'affiche la carte des randos, un point bleu marque ma position.

camillemonchicourt commented 5 years ago

Même problème avec une 3.1.0 sur un Android 9. Rien ne se passe quand on clique sur Trier par distance.

APK de test Rando Écrins V3 : https://drive.google.com/file/d/1F3e_0dGcQ83EH6hClg6vS8PVX_-gQtoN/view?usp=sharing

camillemonchicourt commented 5 years ago

Dans la V2, quand les randos étaient triées par distance, l'info était affichée pour comprendre le fonctionnement et avoir une idée de cette distance :

Screenshot_20190705_204723_com makinacorpus randoecrins

Quand on avait la position GPS, les patrimoines et contenus touristiques étaient eux aussi triés par distance avec l'info affichée, très utile quand on parcourt la rando :

Screenshot_20190705_204819_com makinacorpus randoecrins

camillemonchicourt commented 5 years ago

Idem pour le tri et l'affichage des étapes d'une itinérance :

Screenshot_20190705_211050_com ecrins gte

bastyen commented 5 years ago

Pour l'ordre le tri c'est ok ? car ça fonctionne bien lors de mes tests

Si oui il faudrait fermer cette issue et en faire une autre pour l'affichage des distances

camillemonchicourt commented 5 years ago

Sur mon terminal, le bouton de tri par distance ne fonctionne pas. Il ne se passe toujours rien quand on clique dessus.

bastyen commented 5 years ago

Ok je viens de tester, c'est un problème de votre Rando admin / Api. La liste des treks n'a pas de coordonnées Cannot read property 'coordinates' of null

Donc je ferme cette issue, et en créé une nouvelle pour l'affichage des distances

camillemonchicourt commented 5 years ago

OK merci pour l'investigation. Je ne vois pas pourquoi car notre Geotrek-admin est à jour et la synchro mobile aussi. Mais OK, à voir à ce niveau donc.

bastyen commented 5 years ago

Le trek 919357 n'a pas de geometry et stop donc le tri

camillemonchicourt commented 5 years ago

OK, bien vu, la géométrie de cette rando était cassée coté Geotrek-admin, suite à l'ajout d'un tronçon. Je corrige pour la prochaine synchro.

Ce genre de problème peut se reproduire donc on pourrait imaginer une vérification du champs et si il est vide, mettre la rando à la fin.

bastyen commented 5 years ago

C'est bon

865d420

camillemonchicourt commented 5 years ago

Merci

camillemonchicourt commented 5 years ago

Après re-synchronisation, c'est bon en effet. Merci !