You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
seules les ascensions réalisées en tête rentrent dans les statistiques, les graphs et la liste de croix
une voie enchainée plusieurs fois peut s'afficher plusieurs fois dans la liste lorsque triée par date mais ne doit compter qu'une seule fois dans les statistiques (bargraph et donuts)
Je ne sais pas si tout le monde est d'accord avec cette définition de la croix. Sinon on peut ajouter une case à cocher (ou peut-être plutot un parametre au niveau du User) pour choisir entre tête ou tête+moulinette pour les croix.
J'ai fait la modif mais je ne lancerai la pull request que si tu es ok avec la demande.
The text was updated successfully, but these errors were encountered:
Pour le deuxième point je suis d'accord, ça semble parfaitement logique.
Pour le premier je ne suis pas sûr, dans les règles de l'art oui une croix c'est enchainé en tête, mais est-ce que les règles de l'art doivent venir brider la façon dont toi tu veux construire ton carnet de croix. De plus tu le marque comme quoi tu l'as enchainé en moulinette.
À la limite ça pourrait être une option "exclure les voies réalisé en moulinette" qui te laisse le choix
Je suis un peu débordé par un développement côté indoor en ce moment, j'ai du mal à être sur tout les fronts, désolé du temps de réponses ! j'ai même pas eu le temps d'aller lire les PR ...
J'ai fait les 2 pull request sur l'app et l'api.
J'ai pu installer l'api en local et générer une basede données minimale pour faire les tests mais ils ne sont pas exhaustifs.
Je propose deux évolutions :
Je ne sais pas si tout le monde est d'accord avec cette définition de la croix. Sinon on peut ajouter une case à cocher (ou peut-être plutot un parametre au niveau du User) pour choisir entre tête ou tête+moulinette pour les croix.
J'ai fait la modif mais je ne lancerai la pull request que si tu es ok avec la demande.
The text was updated successfully, but these errors were encountered: