Does what it says on the tin...
qt: Add total balance in overview page #2749
pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2013_06_addtotalbalance changing 2 files +40 −3-
laanwj commented at 9:13 AM on June 7, 2013: member
-
Diapolo commented at 9:37 AM on June 7, 2013: none
ACK, nice addition :).
-
qt: Add total balance in overview page cb85aeb06c
-
subSTRATA commented at 2:17 PM on June 7, 2013: none
Total balance would be useful on Transactions tab as well. For example, if one filters transactions to some specific address, date or both address and date, there is no sum function anywhere and there is no Total row at the bottom of the list so it is not easy to quick-check totals.
-
sipa commented at 4:28 PM on June 7, 2013: member
Looks good. Haven't tested or checked the code.
-
laanwj commented at 7:41 AM on June 8, 2013: member
@sipa It's only one line of code change, the rest is ui markup :smile: @subSTRATA yes it could be useful to query the total of subsets of transactions, this is also requested in #627
-
BitcoinPullTester commented at 5:14 PM on June 8, 2013: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/cb85aeb06cfcac0812c56f24ebf429cf0904658b 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 referenced this in commit 8b313c9331 on Jun 9, 2013
- laanwj merged this on Jun 9, 2013
- laanwj closed this on Jun 9, 2013
- laanwj deleted the branch on Apr 9, 2014
- DrahtBot locked this on Sep 8, 2021
Contributors