AshharAhmadKhan commented on PR #6325: URL: https://github.com/apache/fineract/pull/6325#issuecomment-5427275754
> Now we have a working API, lets write an integration or E2E test for it ;) hey adam, regarding the integration/E2E test, I think we should get #6330 merged first. It fixes the EmailDataValidator using the wrong ScheduledEmailConstants, which currently causes valid /v1/email CREATE requests using the actual email API parameters to be rejected. you merged #6331 which fixes the CREATE NPE, so #6330 is the remaining prerequisite for getting a clean end-to-end CREATE → UPDATE flow. Once #6330 is merged, I can add the integration/E2E test on top of this PR. -- 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]
