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: Fix units display on lab ref ranges #2925

Open
1 of 3 tasks
angelathe opened this issue Nov 18, 2024 · 2 comments
Open
1 of 3 tasks

BUG: Fix units display on lab ref ranges #2925

angelathe opened this issue Nov 18, 2024 · 2 comments
Labels
🐛 bug Something isn't working low Low priority issues Streamline eCR Issues for Streamline eCR team to work on

Comments

@angelathe
Copy link
Collaborator

Description

Please update how units are displayed for lab reference ranges. If the unit is %, we want no visible space between the value and the % -- <value>%. For all other cases, units should be separated from the value by a space like <value> <unit>.

To do this, we have to change the FHIR path for observationRefRange. Currently, we pull all the reference range values from the text field. If the low and high value and unit fields exist under referenceRange, we should pull from them instead.

Steps to reproduce

10e5b07b-5d3f-426f-a7e1-13ea909ff5a5: Link to eCR Viewer

Expected Result

Screenshot 2024-11-18 at 14.08.42.png

Actual result

Screenshot 2024-11-18 at 14.03.53.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

No FHIR conversion should be required for this ticket.

@angelathe angelathe added Streamline eCR Issues for Streamline eCR team to work on 🐛 bug Something isn't working labels Nov 18, 2024
@mcmcgrath13
Copy link
Collaborator

@angelathe I remember seeing you merge a PR that I think fixes this - is that the case?

@angelathe
Copy link
Collaborator Author

@angelathe I remember seeing you merge a PR that I think fixes this - is that the case?

It added the units for the value, but this references the ref range.

Here, the units already display, but Ashton wanted it so that there is no space between %, or a space between any other . Made this into a separate ticket bc it was out of scope of the other one

@emmastephenson emmastephenson added the low Low priority issues label Dec 11, 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 low Low priority issues Streamline eCR Issues for Streamline eCR team to work on
Projects
None yet
Development

No branches or pull requests

3 participants