Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix drug order form frequency #2189

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

charity553
Copy link
Contributor

@charity553 charity553 commented Jan 13, 2025

Requirements

  • This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
  • My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
  • My work includes tests or is validated by existing tests.

Summary

Screenshots

Screencast.from.13-12-2024.12.11.03.WB.webm

Related Issue

Other

Copy link
Contributor

@CynthiaKamau CynthiaKamau left a comment

Choose a reason for hiding this comment

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

Thanks @charity553 fir this, it looks like you are fixing the layout and not the frequency. Please fix the conflicts

@charity553
Copy link
Contributor Author

on it @CynthiaKamau

@denniskigen
Copy link
Member

Yeah, it's not super clear what this PR does. Could you annotate it with the appropriate conventional commit label and add a summary at the top, @charity553. We need those for posterity so folks can have a searchable record for why decisions were made in the past.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants