Follow up of #9016
[qt] Return useful error message on ATMP failure #9043
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1611-qtATMPerror changing 3 files +8 −4-
MarcoFalke commented at 5:27 PM on October 30, 2016: member
-
[qt] Return useful error message on ATMP failure 3333e5a1e5
- MarcoFalke added the label GUI on Oct 30, 2016
-
jonasschnelli commented at 7:40 AM on October 31, 2016: contributor
Concept ACK. Will test soon.
-
jonasschnelli commented at 2:00 PM on October 31, 2016: contributor
Any recommendation how to test this?
-
MarcoFalke commented at 2:15 PM on October 31, 2016: member
You could try to create a transaction which violates policy (e.g. a "free" transaction with full mempool or one which violates the ancestor/descendant limit)
src/qt/bitcoin-qt -testnet -limitancestorcount=1 -
jonasschnelli commented at 7:32 AM on November 1, 2016: contributor
Tested ACK 3333e5a1e582225907b201cca64322d7620fbeec <img width="532" alt="bildschirmfoto 2016-11-01 um 08 31 52" src="https://cloud.githubusercontent.com/assets/178464/19882388/af5d869a-a00d-11e6-83d7-4be6e8e91557.png">
- jonasschnelli merged this on Nov 1, 2016
- jonasschnelli closed this on Nov 1, 2016
- jonasschnelli referenced this in commit 273bde37d8 on Nov 1, 2016
- MarcoFalke deleted the branch on Nov 1, 2016
-
MarcoFalke commented at 9:38 AM on November 1, 2016: member
(Strictly speaking, it should be impossible to test. Right now it can only be tested due to bugs such as #9019.)
-
jonasschnelli commented at 9:44 AM on November 1, 2016: contributor
@MarcoFalke: you can test it (=run into such situation) in conjunction with CoinControl.
- codablock referenced this in commit 9a70cb2fc2 on Sep 19, 2017
- codablock referenced this in commit b2229356b5 on Jan 13, 2018
- andvgal referenced this in commit 6561f0ac3b on Jan 6, 2019
- CryptoCentric referenced this in commit 82ced75d47 on Feb 15, 2019
- MarcoFalke locked this on Sep 8, 2021
Contributors
Labels