Skip to content

Mise à jour des profils FHIR suite aux retours#151

Merged
Haoura merged 4 commits into
mainfrom
resolveIssues
Jun 4, 2026
Merged

Mise à jour des profils FHIR suite aux retours#151
Haoura merged 4 commits into
mainfrom
resolveIssues

Conversation

@Haoura

@Haoura Haoura commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Issues traitées :

  • OID → URL (bonnes pratiques FHIR)
  • Bindings & contraintes (statuts, Encounter.class, Condition.bodySite, system)
  • Modélisation FHIR (root/extension, héritages, profils, system)
  • Cohérence des profils & identifiants (AllergyIntolerance, héritages...)
  • Dates & logique métier (détection vs fin)
  • Structure IG & organisation (extensions, dossiers modèles logiques)

Preview

https://ansforge.github.io/interop-IG-document-core/resolveIssues/ig

@Haoura Haoura requested a review from nriss June 4, 2026 12:40

@nriss nriss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Merci beaucoup, beaucoup mieux ! :)

J'ai demandé qq corrections avant de pouvoir merge

* relationship[Role].coding.code = #CON
* relationship[Role].coding.system = "https://mos.esante.gouv.fr/NOS/TRE_R260-HL7RoleClass/FHIR/TRE-R260-HL7RoleClass"
* relationship[Role].coding.display = "Informateur"
* relationship[Role].coding from FRValueSetRelationshipRole (required)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hello :)
Pardon je n'ai peut être pas été clair.
S'il y a un code qui est fixé, on peut effectivement fixer le system. La bonne pratique est d'écrire en une ligne comme ça :

Suggested change
* relationship[Role].coding from FRValueSetRelationshipRole (required)
* relationship[Role] = $TRE_A13-HL7ParticipationType#DIS "Responsable de la sortie"

Les deux ValueSet contenant un seul code peut donc être supprimé.

Les endroits qui me posent problème sont les endroits où juste le system est fixé, comme c'est le cas ici :

* vaccineCode.coding[cis].system = "https://smt.esante.gouv.fr/terminologie-bdpm"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(je vois que ça a été fait plus haut, c'est top ! :)

A refaire également ici, pas besoin de créer un VS pour un seul code

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Le profil FRRelatedPersonDocument.fsh a été mis à jour : le ValueSet a été supprimé et remplacé par :

  • relationship[Role] = $TRE_R260-HL7RoleClass#CON "Informateur".

@@ -0,0 +1,6 @@
ValueSet: FRValueSetUCUMDocument

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A checker, il existe déjà je pense ce valueset, il faut regarder dans les NOS ou dans l'ig terminologie d'hl7

Peut être celui-ci ? https://terminology.hl7.org/en/ValueSet-ucum-units.html

* participant[responsibleParty].type ^short = "Type de participation"
* participant[responsibleParty].type.coding.code = #DIS
* participant[responsibleParty].type.coding.system = "https://mos.esante.gouv.fr/NOS/TRE_A13-HL7ParticipationType/FHIR/TRE-A13-HL7ParticipationType"
* participant[responsibleParty].type.coding from FRValueSetParticipationTypeEncounter (required)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
* participant[responsibleParty].type.coding from FRValueSetParticipationTypeEncounter (required)
* participant[responsibleParty].type from FRValueSetParticipationTypeEncounter (required)

On peut appliquer le binding au codeableconcept directement

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Nous pouvons supprimer ce nouveau binding, puisqu’il ne contient également qu’un seul code, et le remplacer par :
participant[responsibleParty].type = $TRE_A13-HL7ParticipationType#DIS "Responsable de la sortie"
comme proposé plus haut dans FRRelatedPersonDocument.fsh.

@Haoura Haoura merged commit 29745d5 into main Jun 4, 2026
1 check passed
@Haoura Haoura deleted the resolveIssues branch June 4, 2026 15:27
github-actions Bot pushed a commit that referenced this pull request Jun 4, 2026
* Evolution suite aux issues de NRISS 29745d5
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.

3 participants