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.

Sign up for GitHub

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

Jump to bottom

OZ-545 & OZ-596: [Combined] Refactor code to use camel-openmrs-fhir and rewrite routes with Java DSL with Integration Tests #27

Closed
wants to merge 76 commits into from
Closed

OZ-545 & OZ-596: [Combined] Refactor code to use camel-openmrs-fhir and rewrite routes with Java DSL with Integration Tests #27

wants to merge 76 commits into from

Conversation

VaishSiddharth
Copy link
Contributor

@VaishSiddharth VaishSiddharth commented Jun 11, 2024 •
edited
Loading

This PR is a combination of these two PRs
#22
#26

This PR is created so that review process can be easier as both the PRs have dependency on each other.

@VaishSiddharth
[WIP]Add Odoo client
61feeab
@VaishSiddharth
Add Odoo model
8ffa829
@VaishSiddharth
[WIP] Mappers added
91f77cf
@VaishSiddharth
[WIP] Add Odoo camel component
a3dd7bc
@VaishSiddharth
OZ-545: Disable Consumer for Odoo component and code refactoring
6d3298a
@VaishSiddharth
OZ-545: [WIP] Endpoint parsing logic added
08a801b
@VaishSiddharth
OZ-545: [WIP] Patient routes and processor added
4d53fcb
@VaishSiddharth
OZ-545: [WIP] Patient and Encounter routing added
d72ecf0
@VaishSiddharth
OZ-545: [COPY]Erpnext structure duplicated with name refactoring
c79ea08
@VaishSiddharth
OZ:545 Remove unused routes
ec77570
@VaishSiddharth
OZ:545 Segregate routes to folders
c2edd8f
@VaishSiddharth
OZ-545: [WIP] Patient mapper improved
864b8db
@VaishSiddharth
OZ-545: Mappers improved
9f7c6c3
@VaishSiddharth
OZ-545: Dependency added to fix build and test routes
d38b44d
@VaishSiddharth
OZ-545: Fix Partner mapper
da26043
@VaishSiddharth
OZ-545: Partner created from fhir Patient in Odoo
431fc28
@VaishSiddharth
OZ-545: [WIP] Fix to create sales order in odoo
b3f81fe
@VaishSiddharth
OZ-545: Create Sale Order with sale order line
651c30f
@VaishSiddharth
OZ-545: Service Request to odoo sales order line
95e2228
@VaishSiddharth
OZ-545: Odoo Client refactored
a0aa994
@VaishSiddharth
OZ-545: Odoo component & Partner Handler refactored
8eb2f7a
@VaishSiddharth
OZ-545: Added new handlers and refactored code
b310228
@VaishSiddharth
OZ-545: Fix build
6dac2ca
@VaishSiddharth
OZ-545: Refactor Processors
b0a667c
@VaishSiddharth
OZ-545: [WIP] Continue refactoring
90a08b2
@VaishSiddharth
OZ-545: [WIP] Continue refactoring added logic to check duplicate sal…
29ab49a 
…e order line
@VaishSiddharth
OZ-545: Move create and update logic to default processor
f9eba78
@VaishSiddharth
OZ-545: Add deceased Patient logic to cancel sale order
7585ce3
@VaishSiddharth
OZ-545: Refactor MedicationRequestProcessor and ServiceRequestProcess…
ba293ef 
…or, duplicate issue fixed
@VaishSiddharth
OZ-545: Fix Patient Routing
bd7a0e7
@mks-d
Copy link
Contributor

mks-d commented Jun 17, 2024

So you're saying that in any case we would see in the 150+ files changed?

@VaishSiddharth
Copy link
Contributor Author

Yes. Keeping in mind that deleted files are also counted towards Files changed

wluyima
wluyima requested changes Jul 1, 2024
View reviewed changes
Copy link
Contributor

@wluyima wluyima left a comment

Choose a reason for hiding this comment

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

Left several comments

corneliouzbett
corneliouzbett reviewed Jul 4, 2024
View reviewed changes
.gitpod.Dockerfile Outdated Show resolved Hide resolved
corneliouzbett
corneliouzbett reviewed Jul 4, 2024
View reviewed changes
integration-tests/src/test/java/com/ozonehis/eip/odoo/openmrs/BaseRouteIntegrationTest.java Show resolved Hide resolved
test-infra-odoo/pom.xml Outdated Show resolved Hide resolved
test-infra-odoo/src/test/resources/application-test.properties Outdated Show resolved Hide resolved
test-infra-odoo/src/test/resources/docker-compose/docker-compose-odoo.yml Outdated Show resolved Hide resolved
odoo-openmrs/src/main/java/com/ozonehis/eip/odoo/openmrs/routes/MedicationRequestRouting.java Show resolved Hide resolved
odoo-openmrs/src/main/java/com/ozonehis/eip/odoo/openmrs/component/OdooComponent.java Outdated Show resolved Hide resolved
odoo-openmrs/src/main/java/com/ozonehis/eip/odoo/openmrs/component/OdooEndpoint.java Outdated Show resolved Hide resolved
odoo-openmrs/src/main/java/com/ozonehis/eip/odoo/openmrs/client/OdooClient.java Outdated Show resolved Hide resolved
...src/test/java/com/ozonehis/eip/odoo/openmrs/MedicationRequestToSaleOrderIntegrationTest.java Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@corneliouzbett corneliouzbett corneliouzbett left review comments

@mks-d mks-d mks-d left review comments

@wluyima wluyima wluyima requested changes

@rbuisson rbuisson Awaiting requested review from rbuisson

@ibacher ibacher Awaiting requested review from ibacher

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@VaishSiddharth @mks-d @wluyima @corneliouzbett