Abhishek6122008 commented on PR #6618:
URL: https://github.com/apache/fineract/pull/6618#issuecomment-6265289555

   Thank you for all the reviews @galovics :D 
   Addressed: the mapper already clamps via safeMonthDay (that call is in 
Charge.java), a new integration test runs the real mapper and job, isDueOnDate 
is package-private static with one monthly/yearly path, the validator rejects 
an interval of 0, and the extra tasklet tests are gone.


-- 
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]

Reply via email to