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

Transcription NeTEX Tarifs 1.3 #84

Merged
merged 8 commits into from
Jun 4, 2024
Merged

Transcription NeTEX Tarifs 1.3 #84

merged 8 commits into from
Jun 4, 2024

Conversation

ptitfred
Copy link
Contributor

@ptitfred ptitfred commented Apr 30, 2024

Manquent:

  • meilleurs png pour les diagrammes vectoriels (fichiers emf)
  • mise à dispo sur environnement de test
  • revue par le GT7

Voir #63 .

Frédéric Menou and others added 4 commits April 30, 2024 18:57
Certaines images se retrouvent zoomées sans bonne raison.
@ptitfred
Copy link
Contributor Author

ptitfred commented May 2, 2024

Pour les curieuses et curieux, le script utilisé pour itérer sur le markdown de cette PR: https://gist.github.com/ptitfred/be104d1d87292f74cc2ddd10c1e46775

@ptitfred ptitfred marked this pull request as ready for review May 2, 2024 15:08
@thbar
Copy link
Contributor

thbar commented May 3, 2024

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.

Copy link
Collaborator

@prhod prhod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Spécialisation du DROIT A REDUCTION utilisé pour les tarifications de
type « pay-as-you-go », où une fois qu'un certain niveau de consommation
a été atteint dans un intervalle de temp donné, un plafond (tel que
spécifié par une ou plusieurs RÈGLES DE PLA-FONNEMENT) est appliqué, par
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
spécifié par une ou plusieurs RÈGLES DE PLA-FONNEMENT) est appliqué, par
spécifié par une ou plusieurs RÈGLES DE PLAFONNEMENT) est appliqué, par


## GROUP OF SALES OFFER PACKAGES (GROUPE D’OFFRES À LA VENTE)

Un groupement d’OFFREs À LA VENTE)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Un groupement d’OFFREs À LA VENTE)
Un groupement d’OFFREs À LA VENTE


## SALE OFFER ENTITLEMENT GIVEN (DROIT D’ACCES A UNE OFFRE)

DROIT accordé pour utiliser une OFFRE A LAVENTE.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
DROIT accordé pour utiliser une OFFRE A LAVENTE.
DROIT accordé pour utiliser une OFFRE A LA VENTE.


## SALE OFFER ENTITLEMENT REQUIRED (DROIT NÉCESSAIRE POUR ACCEDER A L’OFFRE)

DROIT nécessaire pour utiliser une OFFRE A LAVENTE.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
DROIT nécessaire pour utiliser une OFFRE A LAVENTE.
DROIT nécessaire pour utiliser une OFFRE A LA VENTE.

@ptitfred
Copy link
Contributor Author

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é.

@prhod
Copy link
Collaborator

prhod commented May 24, 2024

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 #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

@ptitfred
Copy link
Contributor Author

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.

@ptitfred
Copy link
Contributor Author

Une autre option serait de déployer la version de la PR #99 en parallèle pour comparer. @prhod seriez-vous intéressé ?

@prhod
Copy link
Collaborator

prhod commented May 29, 2024

@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 ?

@thbar
Copy link
Contributor

thbar commented May 29, 2024

@prhod @ptitfred après relecture de #99 j'aurais été d'avis de faire:

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.

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?

@ptitfred
Copy link
Contributor Author

oui c'est en cascade.

@prhod
Copy link
Collaborator

prhod commented May 29, 2024

ok faisons comme ça

Co-authored-by: Frédéric Menou <frederic.menou@gmail.com>
@ptitfred
Copy link
Contributor Author

@prhod https://deploy-preview-21--transport-normes.netlify.app/normes/netex/tarifs/ est à jour avec les changements de #99.

@TuThoThai TuThoThai self-requested a review June 3, 2024 11:07
@TuThoThai
Copy link
Collaborator

TuThoThai commented Jun 3, 2024

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 !

@ptitfred
Copy link
Contributor Author

ptitfred commented Jun 3, 2024

@TuThoThai https://deploy-preview-21--transport-normes.netlify.app/normes/netex/tarifs/ est à jour avec les changements de #100.

Copy link
Collaborator

@TuThoThai TuThoThai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est validé pour moi ! Un grand merci @ptitfred et @thbar !

@thbar
Copy link
Contributor

thbar commented Jun 4, 2024

@TuThoThai pour expliciter, est-ce qu'on a le go officiel pour publier sur le site dès à présent ?

@TuThoThai
Copy link
Collaborator

@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 !

@ptitfred ptitfred merged commit d394e0b into main Jun 4, 2024
1 check passed
@ptitfred ptitfred deleted the netex-tarifs-markdown branch June 4, 2024 14:15
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.

None yet

4 participants