Skip to content

Horaires - Ajout de ArrivalDayOffset #165

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

prhod
Copy link
Collaborator

@prhod prhod commented Mar 5, 2025

Traitement de l'issue #159

(j'ai laissé le formatage de la ligne en class="odd" pour éviter de refaire tout le tableau, sachant qu'il n'y a pas de modification de style en ligne.

@ArnaudOggy
Copy link

ArnaudOggy commented Mar 7, 2025

Est-tu sur de la modification @prhod ?
La modification porte sur ServiceJourney > passingTimes (de type TimetabledPassingTime, en cardinalité 0:*).

C'est ce type entier TimetabledPassingTime qui n'est pas documenté.
En 6.2.1 c'est le type PassingTime qui est décrit, pas TimetabledPassingTime.

@prhod prhod force-pushed the arrivaldayoffset branch from 0de0df0 to f82aa9c Compare March 10, 2025 09:37
@prhod
Copy link
Collaborator Author

prhod commented Mar 10, 2025

Merci @ArnaudOggy bien vu ! J'ai repris la correction sur PassingTable (en corrigeant les problèmes d'affichage du tableau en même temps).
Pour le fait que ce soit PassingTime et pas TimetabledPassingTime, c'est un sujet plus global de reprise du contenu. Nous venons de lancer ce travail sur les tarifs, je voudrais éviter de multiplier les travaux en cours. Mais oui, c'est à clarifier

@TuThoThai TuThoThai added the Offre tarifaire Pour toute discussion en lien avec NeTEx France - Tarifs label Mar 10, 2025
@TuThoThai TuThoThai added this to the v2.4 milestone Mar 26, 2025
@ArnaudOggy
Copy link

@prhod
Mince, j'étais parti pour approuver la PR, mais il semble que la mise en page ait sauté sur la modification.

Note subsidiaire : n'importe qui est apte à approuver une PR avant qu'elle ne soit mergée (par vos soins) ?
Je n'osais pas le faire par méconnaissance des process, et ne me sentant pas apte à "approuver".

@@ -951,19 +951,16 @@ Pour ***TrainSize*** voir *6.10.1-Train.*

<div class="table-title">PassingTime – Element (objet inclus)</div>

| | | | | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Il est possible que cette ligne soit nécessaire pour le moteur markdown utilisé.

| | ArrivalTime | xsd:time | 0:1 | Heure d'arrivée. |
| | ***ArrivalDayOffset*** | *xsd:integer* | 0:1 | Nombre de jour de décalage par rapport au jour de début de course (permet de gérer les courses à cheval sur plusieurs jours). |
Copy link
Contributor

Choose a reason for hiding this comment

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

"Nombre de jours" ?

| | DepartureTime | xsd:time | 0:1 | Heure de départ. |
| | | | | |
| | ***DepartureDayOffset*** | *xsd:integer* | 0:1 | Nombre de jour de décalage par rapport au jour de début de course (permet de gérer les courses à cheval sur plusieurs jours). |
Copy link
Contributor

Choose a reason for hiding this comment

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

"Nombre de jours" ?

@TuThoThai TuThoThai linked an issue Apr 21, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Offre tarifaire Pour toute discussion en lien avec NeTEx France - Tarifs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Horaires - Question sur TimetabledPassingTime
4 participants