Introduce WalletModel and loadWallet functionality #417
Conversation
|
forgot to push some stuff? |
johnny9
force-pushed
the
wallet-model
branch
2 times, most recently
from
7c99b8e
to
df375c0
Compare
There was a problem hiding this comment.
- On regtest, I generated a wallet and did generatetoaddress 200
Then the balance was displayed like this
QT:
- regularly it is rather slow or not working
(it's not because they are big unused wallets that need to load, it works fine when using QT)
screencastload.webm
- I've had a couple times that after closing the app the
Saving...window stayed on for multiple minutes while there seems to be no reason as there was no heavy usage that needed to be saved, maybe related to previous point
johnny9
force-pushed
the
wallet-model
branch
from
df375c0
to
6d02c05
Compare
johnny9
force-pushed
the
wallet-model
branch
from
6d02c05
to
26b5ef9
Compare
There was a problem hiding this comment.
26b5ef9
the negative balance issue is fixed
- Sometimes it does, but oftenly it doesn't work as supposed
Screencast.webm
- the balance doesn't update when balanceChanged
- the shutting down window shows all the time now (indefinitely, many minutes, have to manually kill it) when shutting down from terminal, doesn't happen when clicking the X of the app
- nit, some amount can be a bit cut off, same is for the wallet name
johnny9
force-pushed
the
wallet-model
branch
from
26b5ef9
to
d155eed
Compare
There was a problem hiding this comment.
ACK d155eed
This is fine as the initial template for the models and controller, balanceChanged is not properly wired.
I would say the idea of loading is not yet finished, we should implement the functionality of loading an encrypted wallet next.
| QString WalletQmlModel::balance() const | ||
| { | ||
| if (!m_wallet) { | ||
| return "0"; |
There was a problem hiding this comment.
this leaves the wallet dropdown in a weird looking state, we'd want to revisit balance anyways on subsequent updates, just noting that this should all change.
| @@ -126,7 +62,7 @@ Button { | |||
| CoreText { | |||
| id: balanceText | |||
| visible: root.showBalance | |||
| text: formatSatoshis(12300) | |||
| text: "₿ " + root.balance | |||
There was a problem hiding this comment.
I guess we'll also need the sat symbol :P
|
Concept ACK. |
When a user selects a wallet from the WalletSelect menu the wallet controller can now load the wallet data in and the name and balance will appear in the WalletBadge
There was a problem hiding this comment.
it sometimes happens that the walletname shown as loaded in the navigationbar is not the same as the wallet shown as loaded/selected in the wallet select dropdown
Screencast.from.20-10-2024.10.25.24.webm
as a result, clicking another wallet doesn't seem to work as it doesn't update the selected walletname in navbar
There was a problem hiding this comment.
tested manually b41a0e4 in -regtest
As soon as bitcoin-qt starts, it seems that loading the latest used wallet is quite slow and takes like 5 secs to show it up on the menu.
Also it's very slow to load/ switch to another wallet (and only non/ 0 balance wallets), for wallets with some balance (using -regtest), doesn't work at all (selected wallet doesn't change so balance either).
Last thing for now, in Qt we have the list of wallets in the file menu to select and load and when there are more then 1 wallet loaded (multi-wallet mode) there's combo box with the loaded wallets to switch between them. I think the designers or Christoph showed me some prototype where there were small icons in the list to indicate which wallet is loaded or not, perhaps it should be another icon/ button to select only the loaded wallets, unloaded or both/ all.
When a user selects a wallet from the WalletSelect menu the wallet controller can now load the wallet data in and the name and balance will appear in the WalletBadge