Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intégration profil SIRI France v1.7 #32

Merged
merged 65 commits into from
Mar 17, 2024
Merged

Intégration profil SIRI France v1.7 #32

merged 65 commits into from
Mar 17, 2024

Conversation

thbar
Copy link
Contributor

@thbar thbar commented Aug 1, 2023

Je crée cette PR surtout pour penser à noter les éléments nécessitant une correction (et j'éditerai ici):

  • Les tableaux markdown (voir 2.4 définition de la structure leader par exemple) ont automatiquement un header, qui est mis en gras par défaut. Plusieurs tableaux du document n'ont pas d'entête, ce qui fait qu'une ligne "normale" est mise en avant en gras, et c'est peu clair.
  • Passer en revue tous les highlights (mark), qui sont volontairement en rouge clignotant pour les repérer. Il faut explicitement les marquer avec la bonne classe CSS que j'introduirai graduellement pour marquer ce qui est spécifique France, spécifique Hub etc.
  • La CSS est en inline pour le moment pour grouper le travail au même endroit (sans cela il faudrait travailler avec un deuxième repository https://github.com/etalab/transport-normes-site)
  • À partir de 5.8.1.1 "Requête StopPointsRequest", les tableaux markdown ont un header vide au dessus. J'ai wrappé les tableaux précédents sans headers avec une <div class="no_h">...</div>, il faudra uniformiser la logique. Je ne sais pas si on réalise un post-traitement ou autre, à voir.

TODOS pour moi

  • TOC qui disparaît à un niveau qui me paraît curieux (Structure EstimatedVehicleJourney, 6.1.3.1.1, h5) - code: https://github.com/etalab/transport-normes-site/blob/main/layouts/partials/toc.html qui devrait faire apparaître jusqu'à niveau h6
  • Références R6, A1 (exemples) à vérifier
  • Tableaux avec des cellules dont la largeur varie bien qu'elles soient dans la même colonne (ex. 2.4)
  • Passer en revue la totalité des tableaux à nouveau pour voir les , , +, :, manquant (voir 3.2.6 Indentation des données) car un grand nombre ont sauté à la conversion
  • Compter les tableaux et voir si on peut simplifier plus ou moins en automatique en markdown ceux qui ne sont pas déjà dans ce format
  • Voir comment déployer une preview sur le site ; il faut probablement faire une PR avec la bonne branche pour le submodule content/normes pour que ça déclenche quelque chose
  • Documenter ci-dessus
  • Expliquer comment j'ai procédé
  • Flèches droite qui manquent "-> VersionCode" en 6.1.3.1 par exemple

@thbar
Copy link
Contributor Author

thbar commented Dec 21, 2023

Réponse de @Henault par email pour le moment:

OK pour publication du profil France v1.7. Le contenu est conforme à la version .doc
Il y a quelques liens internes qui ne fonctionnent pas (je ferai la remarque une fois publié sous Git).
Concernant la forme, grossir les flèches d’indentation serait pas mal si possible.

Je me note les todos suivantes:

  • Renommer le fichier (là on est sur une version side-by-side qui m'a été utile)
  • Corriger le fait qu'on voit de la "CSS" à la place du résumé
  • Reprendre les TODOs ci-dessus, faire le ménage dedans, créer des tickets si nécessaire
  • Faire un rendu de test local et merger

@thbar thbar marked this pull request as ready for review December 21, 2023 09:28
@thbar thbar self-assigned this Jan 23, 2024
@thbar
Copy link
Contributor Author

thbar commented Jan 23, 2024

@thbar
Copy link
Contributor Author

thbar commented Jan 25, 2024

/cc @TuThoThai @Aurige et il me manque Thierry en particulier, je n'ai pas encore son handle GitHub !

Je remets ici ce que j'ai mis par e-mail, c'est une meilleure habitude 😄 pour la transition qu'on opère.

Il faut qu'on avance rapidement sur le merge de ce draft SIRI 1.7, et donc une dernière review de https://deploy-preview-13--transport-normes.netlify.app/normes/siri/profil-france/.

En effet:

  • on a eu encore une demande la semaine dernière de ce document, et les personnes se lancent dans un dév en ce moment même (ce qui est une bonne nouvelle !), et s'appuient... sur la preview !
  • le merge permettra de travailler sur la suite de plein de choses (dont la séparation des repositories pour 1 NeTEx, 1 SIRI, mais aussi la suite des évolutions fonctionnelles)

Pour alléger le "poids mental" de ce merge, je recommande (comme dit également par Christophe) tant que possible de créer un nouveau ticket spécifique avec "tout ce qui n'aura pas été fait dedans" - sauf points vraiment graves qu'on estimerait bloquants.

En effet sinon c'est un phénomène courant sur les PR, elles peuvent traîner facilement etc.

On débloquera ainsi la suite !

❓ si quelqu'un a le handle GitHub de Thierry, pouvez-vous le pinger ici ? Merci ! (surtout vu que c'est lui le relecteur principal si j'ai compris !)

@thbar thbar changed the title [DRAFT] Intégration profil SIRI 1.7 [DRAFT NE PAS MERGER] Intégration profil SIRI 1.7 Jan 25, 2024
@thbar
Copy link
Contributor Author

thbar commented Feb 6, 2024

Poke @Henault, @Aurige, @albanpeignier, @TuThoThai en particulier, j'ai envoyé un email de relance là dessus, il est vraiment souhaitable de finaliser (quitte à noter ce qui ne va pas dans un ticket comme dit précédemment) pour débloquer le reste des tâches, en particulier:

@TuThoThai
Copy link
Collaborator

@thbar
La version du profil France v1.7 de SIRI disponible en preview ici est validée pour publication.
La validation a été faite et confirmée par :

Note : les changements éditoriaux seront fait post-publication car non urgent. Ils sont repris ici.

@thbar
Copy link
Contributor Author

thbar commented Feb 19, 2024

@TuThoThai @Henault merci infiniment ! Je prendrai le relais à mon retour de congés 😄 mais je voulais vous remercier !

@thbar thbar changed the title [DRAFT NE PAS MERGER] Intégration profil SIRI 1.7 Intégration profil SIRI France v1.7 Mar 17, 2024
@thbar
Copy link
Contributor Author

thbar commented Mar 17, 2024

J'ai réalisé quelques correctifs:

  • Ajout explicite de v1.7 dans le titre
  • Modification de la date affichée pour coller à la date réelle de validation du document d'origine
  • Déplacement de la CSS plus bas dans le document pour éviter qu'elle soit affichée en tant que résumé
  • Tests en local

Ça semble concluant, je vais pouvoir vérifier la preview puis merger.

@thbar thbar merged commit 536ce04 into main Mar 17, 2024
1 check passed
@thbar thbar deleted the integration-profil-1.7 branch March 17, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants