-
Notifications
You must be signed in to change notification settings - Fork 5
Mp 1677 #363
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
Mp 1677 #363
Conversation
|
@pieter-edelman-nictiz Zou jij weer een "sanity" check willen doen" |
pieter-edelman-nictiz
left a comment
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.
Hi @hvroege-nictiz,
Ik zie dat je de component twee keer hebt aangemaakt, een keer in Test en een keer in Cert. Is er een reden om niet één setje componenten te gebruiken?
Daarnaast nog een nitpick; in FHIR kan je geen OID gebruiken als .system zoals de tekst nu zegt, je system is een OID-gebaseerde URN.
Hi @pieter-edelman-nictiz Het viel mij ook op dat de _component map in Cert en Test bestaat. Dit is 2 jaar geleden zo ingericht, dus de reden voor deze keuze weet ik niet. Ik zal de tekst aanpassen naar oid-based URN, maar dit zal dan misschien ook op andere plaatsen moeten gebeuren (https://informatiestandaarden.nictiz.nl/wiki/mp:V3.0.0_MedicationProcess9_FHIR#Resource_identification). |
De plek van de components-folder binnen de projectfolder is ook maar gewoon historisch zo gegroeid, maar er is wel een parameter om dat anders in te stellen. Je kan bv. bij eOverdracht spieken hoe dit anders georganiseerd kan worden.
Het is een technicality, maar ja ... correct is correct. |
De testuitkomst is nu zoals Lauri het verwachtte in https://nictiz.atlassian.net/browse/NICTIZ-28786