This repository has been archived by the owner on May 7, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 113
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kckeim
added
R4
Applies to R4 FHIR resources
DSTU2
Applies to DSTU2 FHIR resources
labels
Oct 27, 2023
amcginn
requested review from
gmodrogan,
krycunn,
RicardoCrespo-ora,
kckeim,
cldeitrich and
summerlcollins
October 30, 2023 16:42
RicardoCrespo-ora
approved these changes
Nov 2, 2023
Someshnsn
approved these changes
Nov 13, 2023
kckeim
approved these changes
Nov 17, 2023
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.
Unchanged, so I couldn't flag it in-context: please HTTPS-ify the field_name_base_url in dstu2's person.yaml.
Current:
field_name_base_url: http://hl7.org/fhir/DSTU2/person-definitions.html#Person
Beyond that, a couple comments is all. Thanks for being patient while I took forever to get here. Nothing from my side that would stop this from merging.
amcginn
commented
Nov 17, 2023
cldeitrich
approved these changes
Dec 15, 2023
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.
OHAIUAD-204315: Approved for style.
SukeshKonjeti
added a commit
that referenced
this pull request
Feb 2, 2024
* Updated MedicationAdminstration R4 Resource (#1010) * Updated MedicationAdministration DSTU2 Resource (#1011) * Review and Updates to Goal (#1027) * Review and Updates to Communication (#1022) * dstu2 Practitioner updates (#1014) * R4 Practitioner updates (#1015) * Person updates for DSTU2 and R4 (#1009) * StructureDefinition updates (#1012) * Updated Medication Order DSTU2 Resource (#1020) * Updated MedicationStatement DSTU2 Resource (#1021) * Updated MedicationRequest R4 Resource (#1019) * Updated Provenance R4 Resource (#1018) * DSTU 2 Patient updates (#1023) * R4 Patient formatting and standard updates (#1028) * R4 Observation,DocumentReference Documentation Updates (#1031) * R4 Encounter Period StartDateTime Sort Update (#1033) * updated Location type search parameter and examples for physicalType (#1034) * [R4][DiagnosticReport][DocumentReference] support search by _lastUpdated (#1032) * [R4][Immunization] Support search by _lastUpdated (#1029) * R4 Binary Autogen CCD add encounter parameter (#1035) * [R4][Immunization] Documentation Update (#1036)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
DSTU2
R4
identifier
PR Checklist