[CoinControl] Allow non-wallet owned change addresses #9413

pull jonasschnelli wants to merge 1 commits into bitcoin:master from jonasschnelli:2016/12/qt_cc_change changing 1 files +13 −0
  1. jonasschnelli commented at 9:27 AM on December 23, 2016: contributor

    Right now, CoinControl does only warn if the custom change address is not owned by the wallet. Sending is still possible. It will silently take another non-custom change address.

    This PR will allow custom non-wallet-owned change addresses after accepting a warn-message-box.

  2. jonasschnelli added the label GUI on Dec 23, 2016
  3. jonasschnelli commented at 9:27 AM on December 23, 2016: contributor

    Should fix #9410.

  4. gmaxwell commented at 10:30 AM on December 23, 2016: contributor

    "Are you sure you want to use this – to the wallet unkonwn – change address?

    suggested: "The address you selected for change is not part of this wallet. Any or all funds in your wallet may be sent to this address. Are you sure?"

    (as an aside, it took me a moment of thought to realize why you thought it was dangerous-- so making the warning explicit my reduce the risk.)

  5. jonasschnelli force-pushed on Dec 23, 2016
  6. jonasschnelli commented at 2:58 PM on December 23, 2016: contributor

    @gmaxwell: Thanks. Switched to your recommendation. Also made sure that the custom change address input field gets cleared when the user did reject the confirmation-message-box.

  7. [CoinControl] Allow non-wallet owned change addresses 54f80263fc
  8. jonasschnelli force-pushed on Dec 23, 2016
  9. gmaxwell commented at 6:10 PM on December 23, 2016: contributor

    Concept ACK.

  10. gmaxwell commented at 7:13 PM on January 5, 2017: contributor

    utACK.

  11. jonasschnelli merged this on Jan 5, 2017
  12. jonasschnelli closed this on Jan 5, 2017

  13. jonasschnelli referenced this in commit fd7d8c7b35 on Jan 5, 2017
  14. MarcoFalke commented at 9:05 PM on January 5, 2017: member

    Post merge ACK

  15. codablock referenced this in commit 9e62719ccf on Jan 18, 2018
  16. andvgal referenced this in commit ef75cf3664 on Jan 6, 2019
  17. CryptoCentric referenced this in commit 4649e15ee5 on Feb 26, 2019
  18. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-16 03:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me