Saifulhuq01 commented on PR #5465: URL: https://github.com/apache/fineract/pull/5465#issuecomment-3867427693
> Thanks for the clear breakdown, > > @Saifulhuq01! The Phase 1 implementation looks solid. > > I have started reviewing the changes—specifically the SavingsAccountForceWithdrawalTest.java and the new permission logic—to ensure my implementation of Phase 2 (Reason Codes & Notification triggers) aligns perfectly with this structure. > > I'll be ready to open the Phase 2 PR once this is merged. @ayushscodes-cs Thanks for the review. Please focus your checks on `SavingsAccountForceWithdrawalTest.java` and the `FORCE_WITHDRAWAL_SAVINGSACCOUNT` permission logic. This architecture defines the boundary for the Core Engine. Ensure that your Phase 2 implementation (Reason Codes/UI) consumes these configurations strictly as defined here, without altering the underlying transaction processing flow. -- 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]
