AshharAhmadKhan commented on PR #6344: URL: https://github.com/apache/fineract/pull/6344#issuecomment-5455206612
> @AshharAhmadKhan Thank you for your interest in this, but one of my DEV is already working on this, i would like to see what the diff between the two PRs Hey @adamsaghy, checked PR #6347. The main fix is identical to mine, same guard clause. The difference is #6347 also adds a second layer of protection in ProgressiveEMICalculator, plus two integration tests that reproduce the actual production scenario. I've only done unit tests on mine. Happy to close this one once #6347 is out of draft and passing, or you can decide either way, just let me know. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
