I created the review to make sure Lukas and you are aware of the auto-commit configuration changes. I will close this review.

I created the review to make sure Lukas and you are aware of the auto-commit configuration changes. I will close this review.

OLMIS-5598: enable subscriptions in openlmis-hapifhir
OLMIS-5598: enable subscriptions in openlmis-hapifhir
No. There never was CsvColumn Builder. FileColumnBuilder was added in this commit.

No. There never was CsvColumn Builder. FileColumnBuilder was added in this commit.

Thank you Nikodem Graczewski and Łukasz Lewczyński for reviewing this. I created a new review for a commit that affects the same files. The new review is for another refactoring that I did to appl...

Thank you Nikodem Graczewski and Łukasz Lewczyński for reviewing this.

I created a new review for a commit that affects the same files. The new review is for another refactoring that I did to apply the feedback I received on the sprint demo. By making it a new review, my hope is that it will be easier to review. I have marked the unresolved issues to resolved because those were resolved in a subsequent commit that went into that next review. I did not want to add that commit to this review, as that would double the effort it takes to review the same work twice.

I have resolved this issue as part of a commit that went into another review. I will mark this feedback as resolved so I can close this review on an obsolete file. https://github.com/OpenLMIS/open...

I have resolved this issue as part of a commit that went into another review. I will mark this feedback as resolved so I can close this review on an obsolete file.

https://github.com/OpenLMIS/openlmis-fulfillment/blob/b2ab87f581421c006cb0270c8a7984f659902450/src/main/java/org/openlmis/fulfillment/service/FileTemplateService.java#L40

This method is removed with a subsequent commit. I am going to resolve this issue here. https://github.com/OpenLMIS/openlmis-fulfillment/blob/b2ab87f581421c006cb0270c8a7984f659902450/src/main/java...
Point taken. I have done another round of refactoring to implement the feedback I received on the sprint demo. Instead of adding those changes to this review, I have created a new review. Hope that...

Point taken. I have done another round of refactoring to implement the feedback I received on the sprint demo. Instead of adding those changes to this review, I have created a new review. Hope that makes it easier.
The new review will have a few more additional tests so the commit could pass sonar analysis. The tests are not for new code. Those are for old code just written to satisfy sonar's 65% branch coverage quality gate. Hope it is okay to keep those in the same commit.

OLMIS-5581: Rename CsvFileTemplate to FileTemplate, increased coverage on some classes
OLMIS-5581: Rename CsvFileTemplate to FileTemplate, increased coverage on some classes
https://github.com/OpenLMIS/openlmis-contract-tests/commits/af7b218619c08df7060ae3e45c2a7f43a97400a3
Working on this.

Working on this.

ref distro is updated. no contract test is affected with this change.

ref distro is updated. no contract test is affected with this change.

Yes. I have updated these. https://github.com/OpenLMIS/openlmis-deployment/commit/66906d3a796cb22bd1e1ba913a1933fd577c257a
OLMIS-5479: Refactor orderFileTemplate to csvFileTemplate to accomodate shipment file template as...
OLMIS-5479: Refactor orderFileTemplate to csvFileTemplate to accomodate shipment file template as...
are these changes due in the service-template repository as well? if the service-template was up to date, we would not have to do this again for another service?

are these changes due in the service-template repository as well?
if the service-template was up to date, we would not have to do this again for another service?

As long as the referencedata service can access resources in hapifhir, successfully authenticating, one seems enough to me.

As long as the referencedata service can access resources in hapifhir, successfully authenticating, one seems enough to me.

I agree. At the moment, the HAPI FHIR server is only configured to support DSTU3. mCSD is a profile of DSTU3. mCSD is not supported on anything below DSTU3.

I agree. At the moment, the HAPI FHIR server is only configured to support DSTU3. mCSD is a profile of DSTU3. mCSD is not supported on anything below DSTU3.