zhangxinyao88 commented on PR #394:
URL:
https://github.com/apache/fineract-backoffice-ui/pull/394#issuecomment-5364833603
I also checked the client transaction undo flow locally after this was
merged.
### Before
_[Screenshot placeholder: native browser confirmation]_
Before this change, the action used `window.confirm('Undo this
transaction?')`.
### After
_[Screenshot placeholder: shared application confirmation dialog]_
With this change, the dialog shows the transaction ID, amount, date, and
explains that the transaction remains marked as reversed.
Behavior check:
- Cancel sent no undo request.
- Confirm sent one undo request.
- After confirmation, the list reloaded and Undo was disabled for the
reversed row.
--
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]
