Comes another opportunity for me to update my blog with the experience i have had in the past week as i was working on my GSoC project. Here's the Coding Period Week 01 blog in case you missed it. I will breakdown my writing today into the various tickets/issues i encountered throughout this week. I was lucky to have some of work merged this week and it felt awesome 😍.
toOpenmrsType translators should set uuid to id part of id, not full: This ticket was created to fix the bug that was on a couple of the resources. This would cause an error where the uuid provided would exceed the maximum length which is 38. For example when i tried doing a patch operation on the patient resource, the error i got was failed to validate with reason: uuid: This value exceeds the maximum length of 38 permitted for this field. This was fixed and merged into master to cater for all other resources it could be affecting.
Add support for Json Patch operations - Patient Resource: This was meant to introduce the json patches to work along with the json merge patches as i explained in the previous blog. This work was also merged into master and it was able to unblock all the other tickets that would wish to implement any of the two patch formats. The rest of the implementation can be at this commit.
Add Support for Json Merge Patch - Service Request Resource: This was meant to introduce json merge patch operations to the Service Request resource. However, this resource was not fully translated into the OpenMRS data model. Hence after spending hours on it, my mentor advised that i created a ticket for it but that could be out of scope for this project. So for future work on the module i ticketed the required changes at https://issues.openmrs.org/browse/FM2-591. This therefore means that the Service Request resource could also be out of scope for this project unless towards the end i get sometime at hand to dive into fully translating the resource.
Add support for Json Merge Patch operations - Medication: This was meant to introduce json merge patch operations to the Medication Resource. The pull request has been sent in and it is await review and hopefully merging into master. While working on this, i realized that the OpenMRS fhirIG index page didn't cater for the Medication resource so that it is easier to locate on the IG so i pushed a followup commit to ensure it does that and it is awaiting review and merging.
This was pretty much it from this past week and i am hoping this week will be more of review of the many pull requests i have created and having them merged. I also intend to work on the integration tests to test out the json patch functionality on the resources.
Thanks for taking your time to through, cheers 🍸🍷
No comments:
Post a Comment