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.
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
feat(#114): Create OpenMRS Mediator #115
base: main
Are you sure you want to change the base?
feat(#114): Create OpenMRS Mediator #115
Changes from 14 commits
5e4451f
4e3de0f
534b1fd
50a3f3f
494ae3d
8c3c4fe
6ed106d
76e9012
ffaa7e0
9b7978d
ac83b03
1fb1351
d9e1c13
449de72
613bf6b
6d03a4f
aba7c95
fda229f
f7a834d
dfe749d
6c3ff27
3eab817
0e38c83
d05968e
70aee07
6141ce8
5bb2eae
12e2a53
88f1014
7a5243c
3db5945
82e29ec
ee8e8dd
6d2c7d2
e91be25
c1cc023
f325edd
7470359
a0fc96b
86bdc31
59f607b
1a5be52
d6db1d6
7d279c1
d5b0252
ef558a2
d8cb295
e3f1a4c
dfe2107
59bf494
0f7ef52
61ccd62
5be2b00
8abfa10
3cc6a55
3216ce7
71df8de
1ec5181
670f3a5
0bca7ae
dcb5e61
40716e8
1de75e4
c9354ff
7b89788
f361ca4
9290c0f
85d0484
539f03e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@witash, can you please add the configs to cht-config so that we can reproduce the entire workflow in the e2e tests using only this repo and not depending on config-gandaki?
Let me know if I can help.
Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the config be private @binokaryg, as is the case with most of our projects?
@lorerod is there an alternative to testing in case the config cannot be copied in a public repo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the config is private with limited access to collaborators.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrablaj, from what I understand, we do not need the entire config. It's just an outbound push configuration and a couple of forms that do not need to be precisely the same. I’m right, @witash?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yea, for the LTFU the configurator put the outbound push configuration in the settings automatically.
What we need to add/modify for tests is not anything specific to the gandaki config. a sample outbound push for cht form submission and inbound form for patient creation and inblound forms
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added the inbound forms and outbound for patient creation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, with this, can I create a patient from CHT to Openmrs and vice versa?