Skip to content

Conversation

@imramkrishna
Copy link

@imramkrishna imramkrishna commented Oct 23, 2025

  • Implement template selection system in new form modal
  • Add comprehensive discharge summary template with medical sections
  • Create reusable form template utilities
  • Update TypeScript and webpack config for template support

This allows users to quickly create discharge summary forms from a predefined template instead of building from scratch.

Fixes #554

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

Related Issue

Other

- Implement template selection system in new form modal
- Add comprehensive discharge summary template with medical sections
- Create reusable form template utilities
- Update TypeScript and webpack config for template support

This allows users to quickly create discharge summary forms from a
predefined template instead of building from scratch.

Fixes openmrs#554
@gracepotma
Copy link
Contributor

Sounds very interesting! Before we assign a reviewer: Can you please kindly add some screenshots or a screen recording to show what the work results look like on your local?

@NethmiRodrigo
Copy link
Collaborator

NethmiRodrigo commented Oct 27, 2025

Thanks @imramkrishna! But, and I hope this doesn't come across as too opinionated, the issue with this is that we are tying the form builder to have inbuilt form templates, which is what we have content packages for essentially, and people could use the import feature. I'm not very hot about having specific json forms hard coded inside the form builder, because
a. We are going to have to maintain them
b. The template list can grow and maintaining it alongside the code for the form builder doesn't seem very optimal to me

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.

Discharge summary

3 participants