GUI: Rephrase Bech32 checkbox texts, and enable it with legacy address default #13251

1. 
   fanquake commented at 2:03 AM on May 17, 2018: member

   • "Bech32" isn't very user-friendly; used "native segwit" as in #11937.
   • You don't spend from addresses.
   • No reason to block off Bech32 access with legacy address default.

   Rebased from #12208

2. GUI: Rephrase Bech32 checkbox text/tooltip

   - "Bech32" isn't very user-friendly
   - You don't spend from addresses

   7ab1c6f6a7
3. GUI: Allow generating Bech32 addresses with a legacy-address default 82dda6bed9
4. fanquake added the label GUI on May 17, 2018
5. fanquake added the label Needs backport on May 17, 2018
6. fanquake added this to the milestone 0.16.1 on May 17, 2018
7. 
   fanquake commented at 2:03 AM on May 17, 2018: member

   cc @jonasschnelli @sipa @TheBlueMatt

8. fanquake cross-referenced this on May 17, 2018 from issue GUI: Rephrase Bech32 checkbox texts, and enable it with legacy address default by luke-jr
9. 
   TheBlueMatt commented at 3:07 PM on May 17, 2018: contributor

   utACK 82dda6bed971c5638962442b4927845fe5eb6600

10. 
    MarcoFalke commented at 4:06 PM on May 17, 2018: member

    utACK 82dda6bed971c5638962442b4927845fe5eb6600

11. MarcoFalke merged this on May 17, 2018
12. MarcoFalke closed this on May 17, 2018

    ---

13. MarcoFalke referenced this in commit ef0e5cd517 on May 17, 2018
14. 
    promag commented at 4:11 PM on May 17, 2018: member

    utACK 82dda6b.

15. 
    GreatSock commented at 4:12 PM on May 17, 2018: contributor

    I think a better tooltip would be "Native segwit addresses (Bech32) reduce your transaction fees later on and offer better protection against typos, but are unsupported by old wallets. When unchecked, an address compatible with older wallets will be created instead."

    But otherwise utACK https://github.com/bitcoin/bitcoin/pull/13251/commits/82dda6bed971c5638962442b4927845fe5eb6600

16. 
    jonasschnelli commented at 4:15 PM on May 17, 2018: contributor

    Post Merge utACK 82dda6bed971c5638962442b4927845fe5eb6600

17. 
    luke-jr commented at 5:59 AM on May 18, 2018: member

    Thanks. Sorry I've fallen so far behind on rebases. :(

18. Bushstar cross-referenced this on May 18, 2018 from issue commits from bitcoin/master by Bushstar
19. MarcoFalke cross-referenced this on May 23, 2018 from issue Qt: Setting for deciding address type (legacy, p2sh or bech32) by hsjoberg
20. fanquake added this to the "For backport" column in a project

---

21. MarcoFalke referenced this in commit ea487f9f90 on May 24, 2018
22. MarcoFalke referenced this in commit 0eda98d01b on May 24, 2018
23. fanquake deleted the branch on May 26, 2018
24. fanquake removed this from the "For backport" column in a project

---

25. fanquake removed the label Needs backport on May 28, 2018
26. 
    fanquake commented at 10:13 AM on May 28, 2018: member

    Backported in #13319

27. HashUnlimited referenced this in commit 85caf24632 on Jun 29, 2018
28. HashUnlimited referenced this in commit ff4de6e1c6 on Jun 29, 2018
29. ccebrecos referenced this in commit 11c054993c on Sep 14, 2018
30. TheArbitrator referenced this in commit 47013b051d on Jun 21, 2021
31. bitcoin locked this on Sep 8, 2021

Contributors

fanquakeTheBlueMattMarcoFalkepromagGreatSockjonasschnelliluke-jr

Milestone
0.16.1