-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes according to the issue. Edunvalvoja issue still unresolved. #9
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some discussion about describing the slices.
input/fsh/patient.fsh
Outdated
* meta.security ^slicing.discriminator.type = #value | ||
* meta.security ^slicing.discriminator.path = "system" | ||
* meta.security ^slicing.rules = #openAtEnd | ||
* meta.security ^slicing.description = "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pitäisikö tähän laittaa jokin kuvaus asiasta? Selkokielisesti siis mistä on kyse. Ja voisi myös viitata siihen tanskalaisten speksiin, http://build.fhir.org/ig/hl7dk/dk-core/StructureDefinition-dk-core-patient.html#use-of-security-label-on-patients. Ihan vain huomiona, että tätä tapaa käytetään laajemminkin.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tanskan mallilla tuo kuvaus on toteutettu erillään resurssista. linkki
Ja jos ihan tarkkoja ollaan niin tanskalaiset eivät määrittele tätä asiaa osana resurssia vaan pelkästään kirjallisena ohjeistuksena. Tein sen nyt noin, koska nyt profiili ainakin ohjaisi mallintamaan asian ohjeiden mukaisesti. Tanskalaisilla on tosin varmaan ollut hyvä syy olla tekemättä tätä näin.
linkki
input/fsh/patient.fsh
Outdated
* identifier ^slicing.discriminator.type = #value | ||
* identifier ^slicing.discriminator.path = "system" | ||
* identifier ^slicing.rules = #openAtEnd | ||
* identifier ^slicing.description = "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here same as with meta.security
slicing. Should we verbally explain that
When using the Finnish national person identifier, the
identifier.system
SHALL beurn:oid:1.2.246.21
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could also say that in this case the the identifier.type
SHOULD be NI
(https://terminology.hl7.org/4.0.0/CodeSystem-v2-0203.html#v2-0203-NI)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add the some short description into slicing. Although that part of the code describes slicing rules so mention about indentifier.type could be explained same way as Turvakielto.
I created a separate pull request for adding the auto-generated scripts for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great progress!
Issues regarding Edunvaloja and organization as contact point are still unresolved. Both may require extending original value sets and need more examing.