Skip to content

Conversation

@nriss
Copy link
Contributor

@nriss nriss commented Dec 15, 2025

Description des changements | Description of changes made

Ajout suite à la demande de @flachalESEA

Preview

https://interop-sante.github.io/hl7.fhir.fr.core/nr-add-numero-am/ig

@nriss nriss requested review from ypoiron and zilliw December 15, 2025 10:50
* identifier ^slicing.rules = #open
* identifier ^slicing.description = "Slice based on the identifier.system value"

* identifier contains numeroAm 0..*
Copy link
Collaborator

Choose a reason for hiding this comment

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

est-ce qu’un PractitionerRole peut avoir plusieurs numéros AM ? Souvent c’est 0..1 ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

On pourrait imaginer qu'un PS a plusieurs métiers (ex quelqu'un qui serait MK et IDE), il est probable que dans ce cas le PS ait plusieurs numéros ADELI (j'en sais rien, seule l'AM le sait).
Toutefois, dans ce cas précis, le PS aura plusieurs PractitionerRole non ? (et la réponse est donc cardinalité à [0..1] )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Le PractitionerRole correspond à la situation d'exercice (IDE dans un établissement donné, Généraliste dans un cabinet précis, ...)
Par intuition, j'ai tendance à dire qu'il y a qu'un numéro AM par situation d'exercice, mais j'en suis pas certain

Je pose la question et j'adapte en fonction !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Passé en 0..1!


* identifier contains numeroAm 0..*

* identifier[numeroAm] ^short = "Identifiant d’activité propre à l’Assurance Maladie. format: 9 digits. synonyme: numeroAM"
Copy link
Collaborator

Choose a reason for hiding this comment

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

si on garde “9 digits”, éventuellement ajouter un invariant de format ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

C'est toujours 9 chiffres oui.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai créé une issue pour traiter ce problème de manière plus générale dans un second temps, ça vous convient ?

#266

@nriss nriss requested review from flachalESEA and ypoiron January 12, 2026 09:52
@nriss nriss added this to the 2.2.0 milestone Jan 12, 2026
@nriss nriss merged commit f3e78d1 into main Jan 12, 2026
1 check passed
@nriss nriss deleted the nr-add-numero-am branch January 12, 2026 10:01
github-actions bot pushed a commit that referenced this pull request Jan 12, 2026
* Ajout du numéro améli au profil PractitionerRole

* Define slicing rules for PractitionerRole.identifier

Added slicing rules for PractitionerRole.identifier.

* add numero AM

* Change identifier 'numeroAm' cardinality to 0..1 f3e78d1
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