[gui] script addresses for sending, not signing #12685

pull bill-walker wants to merge 1 commits into bitcoin:master from bill-walker:master changing 4 files +33 −3
  1. bill-walker commented at 9:34 AM on March 14, 2018: none

    pull request to change all dummy addresses to script addresses was a bad idea, since signing with script address is not possible. this PR keeps pubkey addresses for signing example addresses.

    I am unfamiliar with git, which is why I reset this repository.

    If someone else can do this better in C++, then good luck. Perhaps the type of dummy address can be a parameter in the setupAddressWidget function.

  2. script addresses for sending, not signing c390456454
  3. fanquake added the label GUI on Mar 14, 2018
  4. bill-walker commented at 9:52 AM on March 14, 2018: none

    build failed, my C++ is not yet good enough to implement this change.

  5. bill-walker commented at 9:54 AM on March 14, 2018: none

    error: ‘setupScriptAddressWidget’ is not a member of ‘GUIUtil’

  6. laanwj commented at 10:14 AM on March 14, 2018: member

    pull request to change all dummy addresses to script addresses was a bad idea

    I'd prefer to just revert that then?

    This seems incredibly circuitous for something that isn't important in the first place.

    ... when did that get merged anyway?

    Oh this is based on #12599 which wasn't merged. In which case: NACK. Let's just keep the code as it is.

  7. fanquake closed this on Mar 14, 2018

  8. MarcoFalke locked this on Sep 8, 2021
Contributors
Labels

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-21 18:15 UTC

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