(feat) O3-5698: Add visit summary section configuration admin page#311
Draft
BlessedAmrita wants to merge 1 commit into
Draft
(feat) O3-5698: Add visit summary section configuration admin page#311BlessedAmrita wants to merge 1 commit into
BlessedAmrita wants to merge 1 commit into
Conversation
BlessedAmrita
marked this pull request as draft
July 18, 2026 19:33
BlessedAmrita
force-pushed
the
feat/O3-5698-visit-summary-config-page
branch
from
July 18, 2026 19:40
693547c to
429dba8
Compare
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requirements
Summary
Adds a Visit Summary Configuration page under System Administration, in a new
esm-patient-documents-admin-apppackage. Admins can see every visit summary section registered by the patientdocuments module, toggle sections on and off, reorder them, save, and preview the resulting PDF for a given visit.The section list comes from
GET /ws/rest/v1/patientdocuments/visitSummary/sections, so a downstream module's custom section appears automatically with no change here. Saving writes the existing global propertiesreport.visitSummary.section.<key>.enabledand.orderthrough the standardsystemsettingREST resource, so there is no new write API. Order values are renumbered in tens on save to leave room for sections to interleave.Sections the module marks as not toggleable (facility header, patient info, footer) render with a locked toggle and an explanatory tooltip. The footer is additionally pinned to the bottom of the list with its reorder arrows disabled, since it renders as page furniture on every page rather than in the body flow.
Preview saves the current settings and then renders the PDF for the visit UUID entered, matching the per-visit nature of the report. Fetch failures are surfaced distinctly: a missing
Get Global Propertiesprivilege shows a permissions message with no retry affordance, and other failures show a retryable error.Screenshots
admin-page.mp4
Related Issue
https://openmrs.atlassian.net/browse/O3-5698
Parent epic: https://openmrs.atlassian.net/browse/O3-5667
Other
Dependencies: The sections endpoint is not yet merged. This page requires patientdocuments with
GET /ws/rest/v1/patientdocuments/visitSummary/sectionsopenmrs/openmrs-module-patientdocuments#29. Reordering is only reflected in the generated PDF once the stylesheet order fix openmrs/openmrs-module-patientdocuments#32 is also in.