Hallo community I am working on this ticket Log in - OpenMRS Issues The tickets requires Updating the openmrs-api dependency of apache velocity from 1.6.2 (which was released 2011-01-08) to 2.0.
I updated apache velocity to 2.0. I updated specifically the dependencies in the pom files. Due to the upgrade I had to change also some import packages such that the code works well with apache velocity 2.0. All tests seem to be passing for the changes I made. PR: TRUNK-5119 Update apache velocity by suubi-joshua · Pull Request #4310 · openmrs/openmrs-core · GitHub
I would love a code review to see if the changes made are sustainable. @thembo42 @jwnasambu @sharif @dkayiwa