Skip to content

Comments

mapping avec le modèle V3 ne s'affiche plus correctement avec la nouvelle publication#429

Merged
sga-esante merged 8 commits intomainfrom
nr-test-ajout-target
Jul 28, 2025
Merged

mapping avec le modèle V3 ne s'affiche plus correctement avec la nouvelle publication#429
sga-esante merged 8 commits intomainfrom
nr-test-ajout-target

Conversation

@nriss
Copy link
Member

@nriss nriss commented Jul 25, 2025

Description des changements

Tests suite au message de @TristanKEREVAL

autre point sur lequel j'allais te tag sur git, mais il semble y avoir un bug avec la dernière version de IGPublisher et les mapping, ils apparaissent plus dans l'ig dans l'onglet mapping des StructureDef
 
Sur le ror ils ont le même problème
 
https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/main/ig/StructureDefinition-ror-practitioner-mappings.html
RORPractitioner - Mappings - Répertoire national de l’Offre et des Ressources en santé et accompagnement médico-social v0.6.0
 
https://interop.esante.gouv.fr/ig/fhir/ror/0.5.0/StructureDefinition-ror-practitioner-mappings.html
RORPractitioner - Mappings - Répertoire national de l’Offre et des Ressources en santé et accompagnement médico-social v0.5.0
 
entre les 2 versions de l'ig, générés par des versions différentes d'ig publisher on perd le mapping
 

Le problème n'a pas lieu côté annuaire santé

Preview

Location branche :

https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/nr-test-ajout-target/ig/StructureDefinition-ror-location-mappings.html

Location main :

https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/main/ig/StructureDefinition-ror-location-mappings.html

Impact API ROR

@nriss nriss changed the title [TEST] mapping [TEST - DO NOT MERGE] mapping Jul 25, 2025
@nriss
Copy link
Member Author

nriss commented Jul 25, 2025

Validé avec @TristanKEREVAL : il faut mettre une target pour que le mapping s'affiche (@sdemeyANS @sga-esante)

@sga-esante sga-esante self-requested a review July 28, 2025 16:30
Copy link
Contributor

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

Modification du Target en "https://industriels.esante.gouv.fr/sites/default/files/media/document/ROR_ME_V3.0_ModeleExposition_VFD2.1_20231031.pdf"

J'ai appliqué la même modification aux autres profiles possédant un mapping avec le modèle V3 sur l'IG actuellement publiée (Location, HealthcareService, Practitioner et PractitionerRole). A noter que dans l'IG actuelle Organization n'a pas de mapping avec le modèle V3, probablement une lacune à corriger dans une mise à jour suivante.

@sga-esante sga-esante merged commit c1655b3 into main Jul 28, 2025
1 check passed
github-actions bot pushed a commit that referenced this pull request Jul 28, 2025
…elle publication (#429)

* Update RORLocation.fsh

* Update RORLocation.fsh

* Update RORHealthcareService.fsh

* Update RORPractitionerRole.fsh

* Update RORPractitioner.fsh

* Update RORHealthcareService.fsh

* Update RORPractitioner.fsh

* Update RORPractitionerRole.fsh

---------

Co-authored-by: sga-esante <168082709+sga-esante@users.noreply.github.com> c1655b3
@nriss
Copy link
Member Author

nriss commented Jul 29, 2025

Les bonnes pratiques de gestion peuvent être améliorées : l'auto-merge n'est pas une bonne pratique, la bonne pratique est de faire une review systématique, même si celle-ci est courte.
Le titre de la PR (qui est réutilisé pour faire le changelog) est à mettre à jour

@sdemeyANS
Copy link
Collaborator

sdemeyANS commented Jul 29, 2025

Modification du Target en "https://industriels.esante.gouv.fr/sites/default/files/media/document/ROR_ME_V3.0_ModeleExposition_VFD2.1_20231031.pdf"

J'ai appliqué la même modification aux autres profiles possédant un mapping avec le modèle V3 sur l'IG actuellement publiée (Location, HealthcareService, Practitioner et PractitionerRole). A noter que dans l'IG actuelle Organization n'a pas de mapping avec le modèle V3, probablement une lacune à corriger dans une mise à jour suivante.

Pour Organization le mapping existe mais ne fonctionne pas. Bug certainement du au fait qu'on hérite de l'ancien fr.core.
Le problème sera résolu lorsque l'héritage du ROR pourra évoluer
image
En attendant il est commenté

@sga-esante sga-esante changed the title [TEST - DO NOT MERGE] mapping mapping avec le modèle V3 ne s'affiche plus correctement avec la nouvelle publication Jul 29, 2025
@sga-esante
Copy link
Contributor

@nriss J'ai changé le titre de ta PR.
Pour le commit sur l'historique de main, il était déjà propre "mapping avec le modèle V3 ne s'affiche plus correctement avec la nouvelle publication".
Concernant le changelog, il est construit à la main, donc, évidemment on ne met pas dedans [TEST - DO NOT MERGE]

@nriss nriss deleted the nr-test-ajout-target branch July 29, 2025 08:05
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