mastercoin-MSC / masterchest-wallet

Masterchest Wallet
MIT License
15 stars 13 forks source link

Masterchest wallet balance should update while syncing #8

Open ripper234 opened 10 years ago

ripper234 commented 10 years ago

Masterchest.info shows 22 MSC and 20 TMSC over at 1HfXDX3ALapNebQC8stTdd5zK7kiCgvX9n.

However the wallet (still syncing, currently at block 289,821) shows 0 MSC & Test MSC for each of these addresses. Does the wallet only update the balance after syncing is completed? If this is the case, you should display '?' or something there instead of 0 while the balance is being calculated.

Since the initial transactions sending funds to the address happened long ago, at block 289,821 the balance for this address should definitely not be zero.

zathras-crypto commented 10 years ago

Ron, this is by design for now. The wallet will never show any results or allow you to interact with state until it has finished synchronizing. This is because the compute logic is scan all blocks, then parse all transactions. They're not currently done in tandem (ie scan block, process block transactions, scan block rinse repeat) - I'd like to do it that way in future.

ripper234 commented 10 years ago

So how about showing '?' there while syncing with a tooltip "balance will be available after syncing is complete" in the meantime?

I suggest we keep this issue open as a reminder. (You can tag it enhancement)