diff options
author | Wladimir J. van der Laan <laanwj@protonmail.com> | 2019-10-02 15:49:42 +0200 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@protonmail.com> | 2019-10-02 15:49:58 +0200 |
commit | 8d39c636aa2c3be52cc1126e51e8515f33fe40c7 (patch) | |
tree | 9019f72b5dbcbd0ec101bb838e267d2d1823c7a7 /.github/ISSUE_TEMPLATE/bug_report.md | |
parent | ddc4e3c2d6857336a38cee49c47bce3ca49ab224 (diff) | |
parent | addaf8af8268d918973883a304025d40af5a33c1 (diff) | |
download | bitcoin-8d39c636aa2c3be52cc1126e51e8515f33fe40c7.tar.xz |
Merge #16952: gui: make sure to update the UI when deleting a transaction
addaf8af8268d918973883a304025d40af5a33c1 make sure to update the UI when deleting a transaction (Jonas Schnelli)
Pull request description:
`CWallet::ZapSelectTx` removes transactions from the internal model, but leaves the UI in the dark.
Adding a `NotifyTransactionChanged()` should avoid having invalid transactions in the GUI.
Fixes #16950
ACKs for top commit:
fanquake:
ACK addaf8af8268d918973883a304025d40af5a33c1 - tested that this fixes #16950
Sjors:
tACK addaf8a: tested with an unpruned wallet by calling `removeprunedfunds` on an RBF-replaced transaction. It neatly disappears from the UI.
kristapsk:
ACK addaf8af8268d918973883a304025d40af5a33c1 (tested both with and without this change)
Tree-SHA512: 65e8c690847f7499e82c9fef67b60d9aaa63c853732fe7fa7281da33054fcdcd9d24f5b86de71b0827728c25bac8efb7db445863f990304ebfee6fc450620c47
Diffstat (limited to '.github/ISSUE_TEMPLATE/bug_report.md')
0 files changed, 0 insertions, 0 deletions