Skip to content

(feat) O3-5032: Add Patient documents module to RefApp#940

Merged
dkayiwa merged 2 commits intoopenmrs:mainfrom
jnsereko:O3-5032
Nov 19, 2025
Merged

(feat) O3-5032: Add Patient documents module to RefApp#940
dkayiwa merged 2 commits intoopenmrs:mainfrom
jnsereko:O3-5032

Conversation

@jnsereko
Copy link
Contributor

@jnsereko jnsereko commented Sep 17, 2025

Summary

This enhancement adds a Patient Documents module to the OpenMRS Reference Application distribution.

Screenshot

Screenshot 2025-10-15 at 12 55 14

Related Issue

https://openmrs.atlassian.net/browse/O3-5032

@jnsereko jnsereko changed the title Add Patient documents module to OpenMRS Distro ReffApp (feat) O3-5032: Add Patient documents module to OpenMRS Distro ReffApp Sep 17, 2025
@dkayiwa
Copy link
Member

dkayiwa commented Sep 17, 2025

@jnsereko have you created a CI plan for this module?

@jnsereko
Copy link
Contributor Author

jnsereko commented Sep 19, 2025

@jnsereko have you created a CI plan for this module?

Yes, i have created it here and depending on this PR

@dkayiwa
Copy link
Member

dkayiwa commented Sep 22, 2025

@jnsereko is there a reason why this is still in draft?

@jnsereko jnsereko marked this pull request as ready for review September 23, 2025 07:53
@jnsereko
Copy link
Contributor Author

jnsereko commented Sep 23, 2025

@jnsereko is there a reason why this is still in draft?

Thank you @dkayiwa and @ibacher
Could we kindly merge this if all is good so that i can move forward with the work here?

@dkayiwa
Copy link
Member

dkayiwa commented Sep 23, 2025

@jnsereko i mean a bamboo CI plan.

@jnsereko
Copy link
Contributor Author

jnsereko commented Oct 15, 2025

Hey @dkayiwa and @ibacher
This adds the Patient Documents Module to RefApp. Could you kindly review?

This is dependent on

@dkayiwa
Copy link
Member

dkayiwa commented Oct 15, 2025

@jnsereko did you try to run this module locally in an O3 instance and confirm that it starts up without any errors?

@denniskigen denniskigen requested a review from ibacher October 23, 2025 13:19
@ibacher ibacher changed the title (feat) O3-5032: Add Patient documents module to OpenMRS Distro ReffApp (feat) O3-5032: Add Patient documents module to RefApp Oct 23, 2025
@denniskigen
Copy link
Member

Is this good to go, @dkayiwa?

@jnsereko
Copy link
Contributor Author

jnsereko commented Oct 29, 2025

@jnsereko did you try to run this module locally in an O3 instance and confirm that it starts up without any errors?

hey @dkayiwa and @ibacher
The errors i found on testing have been addressed with openmrs/openmrs-module-patientdocuments#3
This also depends on configuration at ~ openmrs/openmrs-content-referenceapplication-demo#48

@dkayiwa dkayiwa merged commit 15ff5e5 into openmrs:main Nov 19, 2025
14 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