AshharAhmadKhan commented on PR #6325: URL: https://github.com/apache/fineract/pull/6325#issuecomment-5458494522
> > > @AshharAhmadKhan #6330 is merged > > > > > > Hey Adam, just wanted to confirm the testing approach before I proceed. > > Would you prefer me to add the CREATE → RETRIEVE → UPDATE → DELETE integration/E2E test directly to this PR, or keep it as a separate PR? > > I was leaning toward a separate PR to keep this one scoped to the handler fix, but happy to add it here if that's your preference. > > Both works to me. I let you decide Hey Adam, I’ll keep the integration test as a separate PR. I’m almost done with it, and I think keeping it separate will prevent scope creep. -- 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]
