-
Notifications
You must be signed in to change notification settings - Fork 1
Issue 56 examples - Hendrik Hartman (F1) and title fixes #97
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
base: develop
Are you sure you want to change the base?
Conversation
|
Doesn't have Hendrik Hartman an Advance directive? Or did I look over it? In the naming excel where does NR stand for? I can imagine that might be confusing for people who look into it? |
Bump validation-outcome-markdown-renderer action to v1.2.2 and simplify SNOMED CT Concept filter in workflow. Update requester reference in CommunicationRequest example to use the correct PractitionerRole resource.
|
I believe this PR is still in draft phase, until Harmke's changes for the other Patient(s) are added too, correct? We should keep in mind that |
It wasn't there before in our R4 examples, but I see we do have it in STU3 so there is content available for an advance directive for Hendrik Hartman. So let's add it for completeness. And fair point about the NR: it is from the WilsverklaringType codelist (meaning Niet Reanimeren verklaring), but I propose we use DNR. |
Yes let's keep it in draft phase. In case there would've been no comments we could've chosen to merge it and let Harmke pick it up from develop, but since there are comments I think it's best from Harmke to work in this branch. And I wasn't aware of the char limitation. If we want to keep some margin, we could change it to Ex1. |
… into issue-56-examples
Example names for patient Hendrik Hartman (originally F1) are changed and artifact titles are fixed so they follow the same naming conventions.
Harmke will fix example names for other patient.