Skip to content

Paramétrisation du nombre maximal de train récupérables#26

Draft
pa-martin wants to merge 26 commits into
Master13011:mainfrom
ProBreizh35:fetch-evolves
Draft

Paramétrisation du nombre maximal de train récupérables#26
pa-martin wants to merge 26 commits into
Master13011:mainfrom
ProBreizh35:fetch-evolves

Conversation

@pa-martin

@pa-martin pa-martin commented May 24, 2026

Copy link
Copy Markdown
Contributor

À ne pas merge tout de suite SVP - j'aimerais qu'on prenne le temps de tester et d'échanger sur les modifications apportées par PB35 et moi-même. J'ai pas mal touché aux devs de PB, notamment sur le front afin de le garder le plus lisible possible.

Contient un ensemble de modifications, je les listerais plus tard, là j'ai un peu la flemme mdr

  • Augmentation du nombre de trains maximal récupérés par l'api (paramétrable)
  • Récupération (automatique) des arrêts de trains sur le trajet
  • Corrections orthographiques mineures
  • Mise en attributs des valeurs "journeys" et "section" du sensor
  • Améliorer / compléter les tests pythons
  • Créer un readme de développement pour expliquer comment lancer les tests et tester le code de manière générale
  • Modification de la carte personnalisée
    • Affichage des X prochain arrêts (X est paramétrable)1
    • Affichage du délai de chaque arrêt
    • Griser les arrêts déjà passés
    • Ajouter de la raison du retard à la carte
    • Créer une erreur personnalisée à la place de TypeError
    • Séparer l'éditeur visuel en sections afin de l'alléger
    • Créer une méthode permettant d'adapter les anciennes configurations

1 Les règles de gestions (RG) du nombre d'arrêts affichables :

  1. S'il y a moins d'arrêt que la limite (X) alors on affiche tous les arrêts
  2. Sinon on affiche les X prochains arrêts
  3. S'il reste moins d'arrêt à venir que la limite alors on prend aussi les arrêts les plus récents

ProBreizh35 and others added 23 commits April 20, 2026 09:33
update, count train before 10 now 20
Rajout des fichiers manquant.
correction apporté pour la vérification : Lint
Quelque changement suplémentaire, surtout sur la carte ou j'ai corrigé le bug de l'émoji qui était affiché en permanance
# Conflicts:
#	README.md
#	custom_components/sncf_trains/www/sncf-train-card.js
Diminution des impacts inutiles et des modifications anodines.
Amélioration du frontend pour être plus flexible
Suppression de la notion de train annulé pour le moment
@pa-martin

Copy link
Copy Markdown
Contributor Author

@ProBreizh35 nos développements mergés
@Master13011 pas mal de modifs apportés, si tu peux prendre le temps de les relire stp.

On essaye de se valider ça d'ici la fin de semaine ?

@pa-martin

Copy link
Copy Markdown
Contributor Author

Ah et pour info, les cards seront cassées : j'ai essayé d'alléger le paramétrage en créant des sections.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants