[Qt] Improve single step in bitcoinamountfield #3614

pull cozz wants to merge 1 commits into bitcoin:master from cozz:cozz2 changing 3 files +15 −7

cozz commented at 7:28 AM on February 2, 2014: contributor

BEFORE: Single step in sendcoins/receive/options dialog: BTC: 100000 satoshis mBTC: 100 satoshi uBTC: 1 satoshi

AFTER: Single step in sendcoins/receive dialog: BTC/mBTC/uBTC: 100000 satoshis

Single step in options dialog: BTC/mBTC/uBTC: CTransaction::nMinTxFee
[Qt] Improve single step in bitcoinamountfield b920148258
BitcoinPullTester commented at 8:04 AM on February 2, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/b920148258ac78d975ded66eebfa8c0c89de3b9b for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj commented at 2:26 PM on February 9, 2014: member

So this makes the step always 100000 satoshis = 1mBTC?

Yes that's a little bit more I consistent suppose.
cozz commented at 3:03 PM on February 9, 2014: contributor

Yes, always 1mBTC for bitcoinamountfield by default.

In options dialog for the fee its been set to CTransaction::nMinTxFee. Otherwise the single step is pretty useless for the fee setting.
laanwj referenced this in commit 7abcd1a1c9 on Feb 17, 2014
laanwj merged this on Feb 17, 2014
laanwj closed this on Feb 17, 2014
DrahtBot locked this on Sep 8, 2021

Contributors

cozz

BitcoinPullTester

laanwj