Skip to content
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

BUG: Encounter Type should use <originalText> over <displayName> #2771

Open
1 of 3 tasks
angelathe opened this issue Oct 16, 2024 · 0 comments
Open
1 of 3 tasks

BUG: Encounter Type should use <originalText> over <displayName> #2771

angelathe opened this issue Oct 16, 2024 · 0 comments
Labels
🐛 bug Something isn't working FHIR conversion required This ticket will need FHIR conversion work low Low priority issues Streamline eCR Issues for Streamline eCR team to work on

Comments

@angelathe
Copy link
Collaborator

angelathe commented Oct 16, 2024

Description

In the example eCRs below, the Encounter Type shows as Ambulatory in the eCR Viewer, but displays as Office Visit in the eICR HTML.

This is because the displayName vs. originalText values of an encompassing encounter name are different values. "Ambulatory" comes from displayName, but "Office Visit" comes from original Text. The eCR Viewer chooses to display the displayName, but the HTML view uses the original Text.

For this ticket, please fix the FHIR conversion so that it prioritizes pulling the encounter type value from <originalText>, and if that doesn't exist, then to pull the information from <displayName>.

IMPORTANT: Please confirm this with Emma/Robert Mitchell/Brady Fausett/etc. first. Making the change above would require changing the conditional order in DataType/_Coding.liquid, and would therefore affect other resources that use this Coding data type.

Steps to reproduce

Example eCRs:

Relevant files:

  • Resource/_Encounter.liquid
  • DataType/_Coding.liquid

Expected Result

The encounter type displayed in the eCR viewer should match the information displayed in the eICR HTML.

Actual result

Example from 4aca74f4-c3b7-49ce-a437-1c8594451926:
image.png

Priority

Provide an estimate for level of impact the bug has today by placing an X in the applicable bracket

Priority

  • Low priority The bug does not have to be fixed immediately. High and medium priority bugs should be addressed first.
  • Medium priority Typically, medium-priority bugs do not affect customers directly and therefore can be fixed in the normal course of testing and development.
  • High priority High priority bugs must be addressed immediately. They often affect the app’s functionality and impact customers and their user experience. They must take priority.

Notes

FHIR conversion would be required for this ticket.

@angelathe angelathe added FHIR conversion required This ticket will need FHIR conversion work Streamline eCR Issues for Streamline eCR team to work on 🐛 bug Something isn't working labels Oct 16, 2024
@emmastephenson emmastephenson added the low Low priority issues label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working FHIR conversion required This ticket will need FHIR conversion work low Low priority issues Streamline eCR Issues for Streamline eCR team to work on
Projects
None yet
Development

No branches or pull requests

2 participants