-
Notifications
You must be signed in to change notification settings - Fork 54
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
(feat) O3-4181 Display Patient Age and Implement View/Edit Results Functionality for Lab Technicians #99
Conversation
@its-kios09 did this design get these designs from zeplin? or does this have approval from the UI/UX team? |
@pirupius I have brought up discussion on slack |
@its-kios09 - let's separate adding the additional patient details from UI enhancements. The UI one will likely take some time to allow for proper designs |
let me revert the enhancements |
@ojwanganto I've completed the update. The latest version is in the GIF video. |
@pirupius, I have reverted the UI to the previous version due to UI/UX concerns. The current changes include the addition of patient details such as patient age and the ability to view and edit lab results. |
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.
Thanks @its-kios09 some minor comments and clean ups to do before we can get this in
@pirupius Hello Pius, I investigated where we could retrieve the patient’s age using the existing hooks. I think I have an idea of doing a custom representation to achieve that |
@pirupius I found an alternative way to retrieve the age from the existing hook by updating the v=full to a custom representation, and it is working well and optimized. however I have updated the PR |
Requirements
Summary
Screenshots
Related Issue
see: https://openmrs.atlassian.net/browse/O3-4181
Other