Skip to content

Comments

Mise à jour des données capacitaires#420

Merged
sga-esante merged 6 commits intomainfrom
cp/maj-donnees-capacitaire
Jul 28, 2025
Merged

Mise à jour des données capacitaires#420
sga-esante merged 6 commits intomainfrom
cp/maj-donnees-capacitaire

Conversation

@sga-esante
Copy link
Contributor

@sga-esante sga-esante commented Jun 30, 2025

Description des changements

  • Modification du scénario 2 add location-supported-capacity (supprimer draft / modification de l'exemple)
  • Ajout d'un scénario 2 BIS replace location-supported-capacity
  • Modification du scénario 3 remove location-supported-capacity (supprimer draft / modification de l'exemple)
  • Modification du scénario 9, passage en déprécié et mise à jour de l'exemple

Preview

https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/cp/maj-donnees-capacitaire/ig pour prévisualiser l'IG d'une branche

Impact API ROR

@sga-esante sga-esante linked an issue Jun 30, 2025 that may be closed by this pull request
@sga-esante sga-esante requested a review from sdemeyANS June 30, 2025 16:29
@sdemeyANS sdemeyANS requested a review from nriss July 16, 2025 12:40
Copy link
Contributor Author

@sga-esante sga-esante left a comment

Choose a reason for hiding this comment

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

Mes retours en commentaire vu ensemble le 17/7.

@sga-esante sga-esante requested a review from sdemeyANS July 17, 2025 09:12
Copy link
Contributor Author

@sga-esante sga-esante left a comment

Choose a reason for hiding this comment

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

Passage en "resolved" des conversations vu ensemble pdt le point du 17/7

@sdemeyANS sdemeyANS added this to the next release milestone Jul 17, 2025
{
"resourceType": "Parameters",
"parameter": [
"parameter": [
Copy link
Member

@nriss nriss Jul 17, 2025

Choose a reason for hiding this comment

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

Peut être fait dans un second temps, mais il serait préférable de créer une ressource FHIR au sein de l'IG plutôt que de l'écrire dans le md.
Ensuite, il y a des outils pour afficher tout ou partie dans le md : https://build.fhir.org/ig/FHIR/ig-guidance/fragments.html
Ca permet notamment de s'assurer que le standard soit bien respecté

**Requête :**
```json
PATCH [BASE]/Location/XXX
PATCH [BASE]/Location?identifier=XXX
Copy link
Member

Choose a reason for hiding this comment

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

A noter, le conditional patch utilisé ici est en draft en R4

Image

Heureusement, je viens de voir, il va être placé en normatif en R6

https://build.fhir.org/http.html#patch

Copy link
Collaborator

@sdemeyANS sdemeyANS left a comment

Choose a reason for hiding this comment

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

Quelques remarques:

  • Attention au patch dans un élément répétable (tableau) et la gestion des index
  • peut être ajouter un warning pour avertir les implémenteurs qu'il faut bien prendre connaissance de la documentation FHIR sur le patch avant d'implémenter
  • préconiser l'utilisation de if-match ? https://hl7.org/fhir/R4/http.html#concurrency

@sga-esante sga-esante merged commit 7fc4d29 into main Jul 28, 2025
1 check passed
@sga-esante sga-esante deleted the cp/maj-donnees-capacitaire branch July 29, 2025 14:46
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.

[CP]: Mise à jour des données capacitaires

3 participants