Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

allow OpenMRS sync to be configureable #138

Closed
witash opened this issue Oct 2, 2024 · 2 comments
Closed

allow OpenMRS sync to be configureable #138

witash opened this issue Oct 2, 2024 · 2 comments
Assignees
Labels
Type: Technical issue Improve something that users won't notice

Comments

@witash
Copy link
Contributor

witash commented Oct 2, 2024

Merging the openmrs branch back to main is currently blocked because the mediator starts the openmrs sync on startup without automatically.

Not all interoperability projects will be OpenMRS, so they should be started based on environment variables or configurable settings.

@witash witash added the Type: Technical issue Improve something that users won't notice label Oct 2, 2024
@witash witash self-assigned this Oct 2, 2024
@witash
Copy link
Contributor Author

witash commented Oct 8, 2024

OpenHIM supports polling channels which matches what we want to do; if enabled, periodically poll an endpoint in the mediator, which does the sync.

  1. move the polling task out of the mediator where it doesn't really belong
  2. rename LTFU mediator as the default CHT mediator (already done)
  3. add a new OpenMRS mediator (already done)
  4. add (back) the sync endpoint for openmrs sync
  5. add a polling channel as a default channel for the openmrs mediator with the sync endpoint as a route

then, enabling the OpenMRS mediator means setting up the openmrs url and credentials in environment variables (or manually creating the channel on an existing openhim setup; the mediator uses channel variables which can have defaults), and enabling the openmrs mediator in openhim by installing or enabling the default channel.

@andrablaj
Copy link
Member

Closed via #139

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Technical issue Improve something that users won't notice
Projects
Status: Done
Development

No branches or pull requests

2 participants