-
Notifications
You must be signed in to change notification settings - Fork 39
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
1868 map family history #2266
1868 map family history #2266
Conversation
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. #1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. metriport/metriport-internal#1868 Signed-off-by: Ramil Garipov <[email protected]>
refs. metriport/metriport-internal#1868 Signed-off-by: Ramil Garipov <[email protected]>
packages/core/src/fhir-to-cda/cda-templates/components/__tests__/make-family-member-history.ts
Outdated
Show resolved
Hide resolved
packages/core/src/fhir-to-cda/cda-templates/components/__tests__/condition-examples.ts
Show resolved
Hide resolved
...e/src/fhir-to-cda/cda-templates/components/__tests__/xmls/family-history-single-relative.xml
Show resolved
Hide resolved
packages/core/src/fhir-to-cda/cda-templates/components/__tests__/family-history.test.ts
Show resolved
Hide resolved
packages/core/src/fhir-to-cda/cda-templates/components/family-history.ts
Outdated
Show resolved
Hide resolved
packages/core/src/fhir-to-cda/cda-templates/components/family-history.ts
Outdated
Show resolved
Hide resolved
packages/core/src/fhir-to-cda/cda-templates/components/family-history.ts
Show resolved
Hide resolved
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.
Left some comments. Main comment is about having a structured value field in the observation within the family member history so there is structured data to parse.
Also, just wanted to check on the status of Epic testing on staging. Can we bump that if its been stalled? If this is the last section, we could also just wait to bump that until we ship this but there will definetly be issues on their end so ....
refs. metriport/metriport-internal#1868 Signed-off-by: Ramil Garipov <[email protected]>
I will construct an updated version of the CDA and ping them again.. |
return "F"; | ||
default: | ||
return "UK"; | ||
} |
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.
second thought. This is supposed to be converting fhir to CDA, but f and m are not valid values in fhir.
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.
and also the name of the function should convey that its converting from fhir to CDA
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.
lol true.. But I'll keep a copy of this function here for future use :P
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.
Approved conditional on resolving last comment
refs. metriport/metriport-internal#1868 Signed-off-by: Ramil Garipov <[email protected]>
Ticket: https://github.com/metriport/metriport-internal/issues/1868
Dependencies
Description
Testing
FamilyMemberHistory
and see that it gets input into CCDRelease Plan