Closed ptitfred closed 4 weeks ago
Pour les curieuses et curieux, le script utilisé pour itérer sur le markdown de cette PR: https://gist.github.com/ptitfred/be104d1d87292f74cc2ddd10c1e46775
Beau boulot, j'ai transmis au GT7 ! Je pense que viser une validation de ça + travail analogue sur #62 avant une release "iso fonctionnelle" serait une belle étape.
@TuThoThai et @Aurige vont parler de tout ça semaine prochaine.
Merci encore et bravo pour la conversion en MD ! J'ai relu jusqu'au chapitre 5 inclus, je valide la confirmité au PDF 1.3 jusque là. J'ai profité de ma relecture pour corriger quelques typos (souvent présentes dans le PDF) en proposition de modifications.
👍
Je m'étais abstenu de corriger les typos évidentes par souci de conformité.
J'ai revu tout le document, je valide la conformité par rapport au PDF 1.3. J'ai repris mes propositions de modifications dans la PR https://github.com/etalab/transport-profil-netex-fr/pull/99 avec quelques autres typos. Dedans il y a quand même une reprise de mise en page de liste à puce qui était faire avec un TAB dans le document word transformé en puce. A voir comment on merge cette PR, avant ou après la release 2.2
Bonjour Pascal,
Merci pour votre review et bien vu pour les soucis de mise en page.
Je propose qu'on merge votre PR #99 dans celle-ci #84 et que je redéploie cette nouvelle version sur l'environnement de test.
Une autre option serait de déployer la version de la PR #99 en parallèle pour comparer. @prhod seriez-vous intéressé ?
@ptitfred Je n'ai pas d'avis tranché sur le sujet. Vu que ce ne sont que des modifications cosmetiques mineurs, je me dis que ça peut être utilisé pour valider le process de mise à jour avec le GT7. @thbar tu en penses quoi ?
@prhod @ptitfred après relecture de #99 j'aurais été d'avis de faire:
Je propose qu'on merge votre PR https://github.com/etalab/transport-profil-netex-fr/pull/99 dans celle-ci https://github.com/etalab/transport-profil-netex-fr/pull/84 et que je redéploie cette nouvelle version sur l'environnement de test.
Si je comprends #99 est une "sister branch" de petites corrections, on l'intègre ici et ça redéploie #84 pour validation finale.
Et après on propose la validation de #84 en GT7?
oui c'est en cascade.
ok faisons comme ça
@prhod https://deploy-preview-21--transport-normes.netlify.app/normes/netex/tarifs/ est à jour avec les changements de #99.
Hello, De mon côté, je valide également la transcription du profil en MD. A noter que j'ai fait des corrections de typo et de syntaxe lors de la relecture, consolidée dans le PR #100. Il faudrait les intégrer pour que le profil publié soit propre. Un énorme merci à @prhod pour la 1ère relecture et à @ptitfred et @thbar pour le passage en MD et publication de la preview !
@TuThoThai https://deploy-preview-21--transport-normes.netlify.app/normes/netex/tarifs/ est à jour avec les changements de #100.
@TuThoThai pour expliciter, est-ce qu'on a le go officiel pour publier sur le site dès à présent ?
@TuThoThai pour expliciter, est-ce qu'on a le go officiel pour publier sur le site dès à présent ?
@thbar, je confirme qu'au nom du GT7 je valide que le contenu de la retranscription du profil NeTEx Tarifs est le même que celui du pdf original (au typos près). C'est donc un go pour publier le profil au format Markdown plutôt que pdf sur le site.
Merci !
Manquent:
Voir #63 .