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

(feat): updated the functionality of the imaging order #12

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

its-kios09
Copy link
Contributor

@its-kios09 its-kios09 commented Dec 16, 2024

Requirements

  • This PR has a title that briefly describes the work done, including the ticket number if there is a ticket.
  • My work conforms to the OpenMRS 3.0 Styleguide.
  • I checked for feature overlap with existing widgets.

Summary

  • Add Age, Gender and clinical diagnosis as part of the patient info on the imaging panel
  • Review of the imaging procedure before approval
  • Printing of the results
  • Updated the UI

Screenshots

imaging

None.

Related Issue

None.

Other

None.

@ojwanganto
Copy link

ojwanganto commented Dec 16, 2024

Thanks so much, @its-kios09. I am wondering whether we can make the report body showing the report and other details a bit bigger and centralized? I just googled and got the below sample. Do you think we can achieve something similar or close?

image

@its-kios09
Copy link
Contributor Author

Thanks so much, @its-kios09. I am wondering whether we can make the report body showing the report and other details a bit bigger and centralized? I just googled and got the below sample. Do you think we can achieve something similar or close?

image

Yes, let me design the report then i will share it with you

Copy link
Contributor

@donaldkibet donaldkibet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kapture.2024-12-17.at.10.25.29.webm

Am getting this error on running this PR

@ojwanganto
Copy link

ping @its-kios09

@donaldkibet
Copy link
Contributor

cc @its-kios09 see the conflicts

@ojwanganto
Copy link

Do we really need the background colors for order status and no instructions? Could we just adopt a UI that's not very shouting? Disclaimer: Am not a UI expert :)
We can discuss this for the next iteration

@donaldkibet
Copy link
Contributor

Sure @ojwanganto, Merging this in

@donaldkibet donaldkibet merged commit e345190 into palladiumkenya:main Dec 17, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants