Skip to content

Passage du champ temporality en optionnal dans le schéma channel#5

Merged
AntoineAugusti merged 4 commits intoetalab:mainfrom
simon-flury:main
Dec 22, 2025
Merged

Passage du champ temporality en optionnal dans le schéma channel#5
AntoineAugusti merged 4 commits intoetalab:mainfrom
simon-flury:main

Conversation

@simon-flury
Copy link
Copy Markdown

@simon-flury simon-flury commented Dec 31, 2024

Bonjour,

Je suis développeur chez Eco-Compteur. Nous proposons des imports et exports en suivant le schéma de comptage des mobilités depuis la plateforme Eco-Visio et depuis l'API d'Eco-Compteur.

Régulièrement, nous faisons face à des channels qui n'ont pas la propriété temporalityrenseignée. Nous vous proposons de la rendre optionnelle.

Merci pour votre retour.

Vous publiez une nouvelle version d'un schéma ?
Pensez à réaliser les actions suivantes.

  • Mettre à jour les fichiers d'exemples
  • Mettre à jour le champ version
  • Mettre à jour le champ lastModified
  • Changer les liens vers les fichiers d'exemples présents dans schema.json et README.md
  • Mettre à jour le fichier CHANGELOG.md en incluant une description de la version
  • Merger cette pull-request
  • Publier un nouveau tag et une nouvelle version
  • Prévenir les utilisateurs de ce schéma

@simon-flury simon-flury requested a review from a team as a code owner December 31, 2024 13:58
@stephane-pignal
Copy link
Copy Markdown

@simon-flury dsl pour le délai, nous regardons cela cette semaine

@thbar
Copy link
Copy Markdown
Contributor

thbar commented Jan 15, 2025

Hello! Comme vu en privé avec @stephane-pignal je commente ici : en tant que co-auteur du schéma et en me basant sur les différents ateliers et le contexte général, ça me paraît un ajustement tout à fait valable.

⚠️ Attention avant de créer une release officielle (qui se fera après le merge de cette PR), il faudra un peu de test en local.

@thbar
Copy link
Copy Markdown
Contributor

thbar commented Jan 15, 2025

Merci @simon-flury pour la contribution, aussi !

Comment thread CHANGELOG.md Outdated
Comment thread channel/schema.json
"created": "2021-05-06",
"lastModified": "2023-05-15",
"version": "0.2.4",
"lastModified": "2024-12-31",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

À modifier de notre côté (@stephane-pignal) avant la release.

353226415,,,300014151,BIKE,,REMOTE TRANSMISSION,MANUAL,OTHER,,OUT,"XXX Vers Angers","Software - PO",PERMANENT,2000-01-01T00:00:00+01:00,2023-04-14T04:00:00+01:00,2023-04-14T04:00:00+01:00,86400,
353226382,,,300014151,BIKE,,REMOTE TRANSMISSION,MANUAL,OTHER,,IN,"Vers Nantes","Software - PO",PERMANENT,2020-08-18T05:00:00+01:00,2023-04-14T04:00:00+01:00,2023-04-14T04:00:00+01:00,86400,
353226417,,,300014151,BIKE,,REMOTE TRANSMISSION,MANUAL,OTHER,,OUT,"Vers Angers","Software - PO",PERMANENT,2020-08-18T05:00:00+01:00,2023-04-14T04:00:00+01:00,2023-04-14T04:00:00+01:00,86400,
353226362,,,300014142,PEDESTRIAN,,REMOTE TRANSMISSION,MANUAL,OTHER,,IN,"Vers Bd Thiers","Software - PO",PERMANENT,2014-11-19T08:30:00+01:00,2023-01-20T04:45:00+01:00,2023-01-20T04:45:00+01:00,86400,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Il y a des lignes avec PERMANENT plus haut donc le fait que c'est autorisé reste bien testé.

@thbar
Copy link
Copy Markdown
Contributor

thbar commented Jan 15, 2025

Petit bug non lié à la PR, qu'il va falloir corriger:

Expected — Waiting for status to be reported

L'intégration continue (GitHub Actions) ne rapporte pas un status "OK", on va devoir corriger ça.

@thbar
Copy link
Copy Markdown
Contributor

thbar commented Jan 21, 2025

Ticket créé ici concernant le souci de PR:

il va falloir traiter ça pour merger sereinement (d'autant plus qu'on a #6 qui est arrivé).

@simon-flury
Copy link
Copy Markdown
Author

Bonjour,
Avez-vous réussi à traiter votre soucis?
Merci pour votre retour

@AntoineAugusti AntoineAugusti merged commit 7b8d0b6 into etalab:main Dec 22, 2025
1 check passed
@AntoineAugusti
Copy link
Copy Markdown
Member

@simon-flury Merci pour la contribution et vraiment navré pour l'attente 😬

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.

4 participants