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

(fix) Updating Imaging Orders and Procedure Orders to require Order Reason #11

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

munania
Copy link
Contributor

@munania munania commented Dec 13, 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

Screenshots

None.

Related Issue

None.

Other

None.

@munania munania changed the title (fix) Updated Imaging Orders to require Order Reason (fix) Updated Imaging Orders and Procedure Orders to require Order Reason Dec 13, 2024
@munania munania changed the title (fix) Updated Imaging Orders and Procedure Orders to require Order Reason (fix) Updating Imaging Orders and Procedure Orders to require Order Reason Dec 13, 2024
Copy link
Contributor

@makombe makombe left a comment

Choose a reason for hiding this comment

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

Thanks @munania LGTM!

@ojwanganto
Copy link

Please share a short video/screenshot showing the outcome.

@munania
Copy link
Contributor Author

munania commented Dec 16, 2024

Please share a short video/screenshot showing the outcome.

Imagiing.vid.mp4
procedure.vid.mp4

@ojwanganto
Copy link

Thanks so much, @munania. Are we able to shift the order reason field above the instructions'? It looks odd where it is.

@munania
Copy link
Contributor Author

munania commented Dec 17, 2024

Thanks so much, @munania. Are we able to shift the order reason field above the instructions'? It looks odd where it is.

Order reason placed above instructions

Screenshot 2024-12-17 002240
Screenshot 2024-12-17 002322

@ojwanganto
Copy link

Thanks, @munania. Over to you @donaldkibet @makombe @Omoshlawi @FelixKiprotich350

@FelixKiprotich350
Copy link
Contributor

Thanks @munania LGTM.

@makombe makombe merged commit 6be2553 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.

5 participants