search

ansforge / IG-fhir-repertoire-offre-ressources-sante

Définition des spécifications de l'API FHIR pour utiliser le Répertoire national de l’Offre et des Ressources en santé et accompagnement médico-social (ROR).
https://interop.esante.gouv.fr/ig/fhir/ror/
MIT License
2 stars 1 forks source link

Sd add MS dans les profils ROR #269

Closed sdemeyANS closed 4 months ago

sdemeyANS commented 6 months ago

Changement voir issue #268 preview https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/ig/sd-add-ms/artifacts.html

sdemeyANS commented 5 months ago

merci :) Un petit commentaire

bien vu :) merci :)

LudivineLacour commented 5 months ago

Relecture de PractitionerRole OK

sdemeyANS commented 5 months ago

Cela paraît logique a priori mais il manque le MS sur les id de chaque ressource. A mon sens cela pourrait créer de la confusion si le MS n'apparaît pas sur ces éléments.

A mon sens MS n'a pas sa place sur l'id. C'est un id logique ('technique') qui n'a pas de valeur métier et n'est pas profilé. On hérite directement du standard FHIR

nriss commented 5 months ago

Cela paraît logique a priori mais il manque le MS sur les id de chaque ressource. A mon sens cela pourrait créer de la confusion si le MS n'apparaît pas sur ces éléments.

A mon sens MS n'a pas sa place sur l'id. C'est un id logique ('technique') qui n'a pas de valeur métier et n'est pas profilé. On hérite directement du standard FHIR

Après discussion avec Sylvain, je m'aligne à son avis! Effectivement, il est préférable de ne pas rentrer trop dans la technique FHIR qui est présente dans la doc officielle : l'usage de l'id est clairement explicitée dans la documentation et n'est pas propre au ROR

sdemeyANS commented 5 months ago

Cela paraît logique a priori mais il manque le MS sur les id de chaque ressource. A mon sens cela pourrait créer de la confusion si le MS n'apparaît pas sur ces éléments.

A mon sens MS n'a pas sa place sur l'id. C'est un id logique ('technique') qui n'a pas de valeur métier et n'est pas profilé. On hérite directement du standard FHIR

Après discussion avec Sylvain, je m'aligne à son avis! Effectivement, il est préférable de ne pas rentrer trop dans la technique FHIR qui est présente dans la doc officielle : l'usage de l'id est clairement explicitée dans la documentation et n'est pas propre au ROR

@LudivineLacour si c'est ok pour toi est-ce que tu peux approuver la PR stp ?

LudivineLacour commented 5 months ago

@sdemeyANS Concernant l'usage des must support, il me semble qu'on s'était dit qu'on ne les mettaient pas uniquement sur les attributs propres au ROR mais aussi sur les éléments natifs hérités sinon cela n'a aucun intérêt. L'objectif est justement pour que l'on puisse visualiser facilement l'exhaustivité des attributs FHIR qui sont implémentés dans l'API du ROR National, y compris ceux hérités.

Est-ce cette logique qui a été réalisée sur les MS ?

sdemeyANS commented 5 months ago

@sdemeyANS Concernant l'usage des must support, il me semble qu'on s'était dit qu'on ne les mettaient pas uniquement sur les attributs propres au ROR mais aussi sur les éléments natifs hérités sinon cela n'a aucun intérêt. L'objectif est justement pour que l'on puisse visualiser facilement l'exhaustivité des attributs FHIR qui sont implémentés dans l'API du ROR National, y compris ceux hérités.

Est-ce cette logique qui a été réalisée sur les MS ?

Hello. Normalement tout ce qui est mappé oui. Quand c'est mappé on ajoute/précise dans la description le contexte d'usage dans le ROR même si on ne change pas la cardinalité. Ensuite ce qui est natif est de facto de cardinalité 1 minimale est à supporter

LudivineLacour commented 5 months ago

Justement je ne comprends pas pourquoi ne pas mettre le MS sur l'id car il est natif mais en cardinalité 0..1 sur les ressources. Uniquement parce que c'est un champ technique ? Je trouve ça un peu étrange et risqué en terme de bonne compréhension de l'IG par les éditeurs

sdemeyANS commented 5 months ago

Justement je ne comprends pas pourquoi ne pas mettre le MS sur l'id car il est natif mais en cardinalité 0..1 sur les ressources. Uniquement parce que c'est un champ technique ? Je trouve ça un peu étrange et risqué en terme de bonne compréhension de l'IG par les éditeurs

Parce que là c'est le standard FHIR. Au delà de l'IG les éditeurs, pour faire du FHIR, doivent regarder la documentation FHIR. https://interop.esante.gouv.fr/ig/documentation/demarrer_sur_fhir.html

sdemeyANS commented 5 months ago

Justement je ne comprends pas pourquoi ne pas mettre le MS sur l'id car il est natif mais en cardinalité 0..1 sur les ressources. Uniquement parce que c'est un champ technique ? Je trouve ça un peu étrange et risqué en terme de bonne compréhension de l'IG par les éditeurs

Parce que là c'est le standard FHIR. Au delà de l'IG les éditeurs, pour faire du FHIR, doivent regarder la documentation FHIR. https://interop.esante.gouv.fr/ig/documentation/demarrer_sur_fhir.html

du coup @LudivineLacour ok pour la PR?