Adds non-mempool wallet balance to overview#911
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process. ConflictsNo conflicts as of last run. |
|
To test, start with from the console. "Non-mempool: " should appear in the Balances pane on the Overview window with the change from the transaction. Restarting with |
029dc0d to
4be5d3d
Compare
4be5d3d to
bc85b4a
Compare
bc85b4a to
026088b
Compare
|
Rebased after merge of parent PR, should be ready to review. Happy to close if some someone wants to take over. |
|
Maybe I'm not understanding the issue, but does this matter? So I am not really getting the issue, we just need to wait for it to be confirmed. IIUC this only makes sense for private-broadcast. |
|
Worst case, I guess: if you make a transaction that doesn't hit the mempool, it won't be visible in your balance, which may lead you to try to make the same transaction again. That might happen if the first transaction was spending some unconfirmed outputs from large transactions, so your new transaction would have exceeded the cluster size limits. Once those large transactions confirm, your nonmempool transaction will be broadcast and able to be confirmed. |

The wallet can contain transactions that are not accepted into the node's mempool (eg due to containing a too large OP_RETURN output, due to too low a feerate, or due to too many unconfirmed ancestors). In the event you end up in this situation, it can appear as if funds have gone missing from your wallet due to the non-mempool balance not being reported. Correct this by reporting the non-mempool balance.
See bitcoin/bitcoin#33671 for further context.