AshharAhmadKhan commented on PR #6144: URL: https://github.com/apache/fineract/pull/6144#issuecomment-5046060876
Hey @adamsaghy While debugging the failing e2e tests, I found the issue wasn't actually in the new undo logic. `reverseTransfer()` was missing a call to `loanTransaction.manuallyAdjustedOrReversed()`, which the other reversal paths already make. I added that, fixed a couple of small expectation issues in the feature file, and now everything passes locally, including all AccountTransfer scenarios and the unit tests. When you get a chance, could you trigger the checks again? Thanks! -- 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]
