Skip to content

OZ-742: Added Bahmni EMR to Ozone.#123

Merged
kipchubett merged 2 commits intoozone-his:mainfrom
Ruhanga:OZ-742
Apr 1, 2025
Merged

OZ-742: Added Bahmni EMR to Ozone.#123
kipchubett merged 2 commits intoozone-his:mainfrom
Ruhanga:OZ-742

Conversation

@Ruhanga
Copy link
Contributor

@Ruhanga Ruhanga commented Nov 22, 2024

@Ruhanga Ruhanga force-pushed the OZ-742 branch 2 times, most recently from b17cca8 to e5f96b0 Compare December 16, 2024 13:34
@Ruhanga Ruhanga marked this pull request as ready for review January 6, 2025 15:02
@rbuisson rbuisson requested a review from kipchubett February 5, 2025 10:18
Copy link
Contributor

@kipchubett kipchubett left a comment

Choose a reason for hiding this comment

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

Thanks @Ruhanga
I've left some comments + a few suggestions.

@kipchubett
Copy link
Contributor

I've tested this with the Ozone PR and below are the issues encountered:

2025-02-06 17:40:17 cp: can't stat '/usr/local/bahmni_config_override/frontend/*': No such file or directory
2025-02-06 17:41:17 cp: can't stat '/usr/local/bahmni_config_override/frontend/*': No such file or directory
  • OpenMRS Service:
ERROR - ModuleFactory.startModules(254) |2025-02-06T14:12:28,345| Module Open-Elis Atom Feed Client cannot be started because it requires the following module(s): org.bahmni.module.bahmnicore Please install and start these modules first.
2025-02-06 17:12:28 ERROR - ModuleFactory.startModules(254) |2025-02-06T14:12:28,361| Module Fhir Terminology Services cannot be started because it requires the following module(s): org.bahmni.module.bahmnicore Please install and start these modules first.
2025-02-06 17:12:28 ERROR - ModuleFactory.startModules(254) |2025-02-06T14:12:28,434| Module Medication Administration cannot be started because it requires the following module(s): org.bahmni.module.bahmnicore Please install and start these modules first.
2025-02-06 17:12:28 ERROR - ModuleFactory.startModules(254) |2025-02-06T14:12:28,442| Module Bahmni IPD cannot be started because it requires the following module(s): org.bahmni.module.medication-administration,org.bahmni.module.bahmnicore Please install and start these modules first.

@Ruhanga
Copy link
Contributor Author

Ruhanga commented Feb 6, 2025

The OpenMRS Service: error, I've seen. This is because the modules for some reason load before others that load later. But is resolved by restarting the service after metadata has completely loaded.

The other two, I've not experienced, however could be related to a dos vs unix encoding issue. I'll be happy to jump on a call to help issues get resolved.

Copy link
Contributor

@kipchubett kipchubett left a comment

Choose a reason for hiding this comment

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

This is already looking good @Ruhanga. Can you resolve the conflicts. I'll be testing the PRs locally and provide more feedback but the changes looks good to me!

@Ruhanga Ruhanga force-pushed the OZ-742 branch 3 times, most recently from cd56baa to 32f6ab0 Compare February 17, 2025 16:33
Copy link
Contributor

@rbuisson rbuisson left a comment

Choose a reason for hiding this comment

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

As agreed, we merge the 3 PRs to support Bahmni EMR in Ozone. The few issues identified will be worked as afterwards.

@kipchubett kipchubett merged commit f4fcea8 into ozone-his:main Apr 1, 2025
2 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