[wallet] remove minimum total fee option #10390

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:killminfee changing 5 files +1 −46
  1. instagibbs commented at 2:38 AM on May 12, 2017: member

    It's of questionable use and complicates further wallet improvements.

  2. remove minimum total fee option 091a9ae21c
  3. gmaxwell commented at 2:42 AM on May 12, 2017: contributor

    We had a minimum total fee? Thats useless.. Concept ACK getting rid of it.

  4. fanquake added the label Wallet on May 12, 2017
  5. MarcoFalke added the label GUI on May 14, 2017
  6. MarcoFalke commented at 11:01 AM on May 14, 2017: member

    Concept ACK. It might be useful in some edge cases, but I don't think this entitles for cluttering the gui and wallet code with this option.

  7. sipa commented at 6:29 PM on May 14, 2017: member

    utACK. Seems untested even.

  8. jonasschnelli commented at 11:32 AM on May 17, 2017: contributor

    Tested ACK 091a9ae21c47ea7c479e58465ff8aaee7a378fc5. <img width="1067" alt="bildschirmfoto 2017-05-17 um 13 30 55" src="https://cloud.githubusercontent.com/assets/178464/26151952/46fecdbe-3b05-11e7-94dd-87c9a0763038.png">

    https://bitcoin.jonasschnelli.ch/build/136

  9. fanquake commented at 3:39 AM on May 19, 2017: member

    utACK 091a9ae Couldn't seem to find any more related code that should have also been removed.

  10. laanwj commented at 6:01 PM on May 22, 2017: member

    utACK 091a9ae

  11. laanwj merged this on May 22, 2017
  12. laanwj closed this on May 22, 2017
  13. laanwj referenced this in commit e4775167cb on May 22, 2017
  14. morcos commented at 8:12 PM on May 22, 2017: member

    woohoo!
    sorry i didn't ACK this

  15. markblundeberg commented at 3:31 AM on February 16, 2019: none

    (note: introduced bug later fixed in #11332)

  16. meshcollider referenced this in commit 4f4ef3138b on Apr 14, 2019
  17. PastaPastaPasta referenced this in commit c2a6f8d095 on Jun 10, 2019
  18. PastaPastaPasta referenced this in commit dc9209bc24 on Jun 11, 2019
  19. PastaPastaPasta referenced this in commit f478e1822d on Jun 11, 2019
  20. PastaPastaPasta referenced this in commit 81f1f5fcfe on Jun 15, 2019
  21. PastaPastaPasta referenced this in commit 91aa37d0ae on Jun 19, 2019
  22. PastaPastaPasta referenced this in commit 92aa4f7a99 on Jun 19, 2019
  23. PastaPastaPasta referenced this in commit a95497d4b6 on Jun 19, 2019
  24. PastaPastaPasta referenced this in commit 460ab2932c on Jun 19, 2019
  25. PastaPastaPasta referenced this in commit b24089984c on Jun 19, 2019
  26. PastaPastaPasta referenced this in commit eb536c9238 on Jun 20, 2019
  27. PastaPastaPasta referenced this in commit 9ca9b4ebd5 on Jun 22, 2019
  28. PastaPastaPasta referenced this in commit e94b853fa1 on Jun 22, 2019
  29. PastaPastaPasta referenced this in commit 76b7e036c0 on Jun 22, 2019
  30. PastaPastaPasta referenced this in commit 5b5bc72ff8 on Jun 22, 2019
  31. PastaPastaPasta referenced this in commit 8946b277ee on Jun 22, 2019
  32. PastaPastaPasta referenced this in commit ae864eb2ce on Jun 24, 2019
  33. barrystyle referenced this in commit f471f78590 on Jan 22, 2020
  34. DrahtBot locked this on Dec 16, 2021
Contributors
instagibbsgmaxwellMarcoFalkesipajonasschnellifanquakelaanwjmorcosmarkblundeberg
Labels
GUIWallet
Linked (view graph)
#10360 [WIP] [Wallet] Target effective value during transaction creation#15557 Enhance `bumpfee` to include inputs when targeting a feerate
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-30 15:15 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me