Confusing design of the wallets and addresses models.

[stdevCrow]

I'm coding the C++ version of the model that shows wallets and its addresses in the main page, and I realize that:

• In the walletsModel.go (

  fibercryptowallet/src/models/walletsModel.go

  Line 50 in fb9e9d3

  type QWallet struct {

  ), there's a QWallet type.

• In the addressesModel.go (

  fibercryptowallet/src/models/addressesModel.go

  Line 41 in fb9e9d3

  type QAddress struct {

  ), there's a QAddress type.

But despite a wallet has a list of addresses, there's no connection between them but a walletId property in the QAddress type, and a QWallet does not have such property, so this whole design makes no sense to me.

Describe the solution you'd like

• A C++ WalletsModel that contains a list of Wallet types that contains an AddressesModel type containing the list of Address types related to that wallet.

It's there any problem with this implementation?

[stdevCrow]

@stdevCrow @olemis pls take a look at this 👆 asap, I need a yes or no to continue.

[olemis]

@stdevCrow I guess model name should be Wallet (instead of Wallets ...)

[stdevCrow]

@olemis Agree, but are you okay with the propossed implementation?

[olemis]

@stdevCrow Original code works that way for a reason , it seems to me that it's due to the fact that golang wallet address objects are generated by wallet enumeration methods hence they might differ from one method call to the next

This would be of particular importance through the C API

[stdevCrow]

@olemis I'm not talking about changing underlying system. I meant that if the addresses contains a walletId, both can be grouped together in the Qt model. The backend will not know the difference. And the model of addresses will not know the existence of the model of wallets. The will work as always, but addresses with the same walletId will be grouped under the same wallet in the WalletModel.

[olemis]

@stdevCrow it seems to me (cmiiw) it's not a wallet ID but an address ID . See BIP-44 hierachical addressing standard . It's not a list of addresses , but an inifinite derivations tree