TheQuantumPhysicist

As discussed in #1497, ideally, the `ColdWalletClient` doesn't implement the `NodeInterface` at all, but in the `NodeController`, only the methods that require communication with the node would require `NodeInterface`.

Currently, errors derived from `thiserror` (especially in the wallet) use `Amount` when errors occur. The problem with that is that the errors appear very unreadable for users, something in the...

Since LMDB uses random access, hard disk drives can have tremendous performance problems. This is how this can be done in Linux: https://stackoverflow.com/questions/908188/is-there-any-way-of-detecting-if-a-drive-is-a-ssd We can do this for all operating...

Unless the wallet is restored from a seed phrase, we shouldn't just scan the whole blockchain when it's empty.

This is considered bad practice, since the decommission key should be in cold storage. - [ ] Wallet CLI - [ ] GUI

The GUI currently supports staking, but not delegations. It makes more sense to support delegations since GUI users are usually less knowledgeable.

Currently, when serializing a block, it's not possible to see the transaction id, because serde serializes fields and doesn't support any modes for serialization to choose how to serialize. This...

Chain pruning

1

Giving that we limit reorgs after some depth, there should be some clean up done in storage to free up space given that it's not used anymore.

Fees of time-locked transactions should not be included in transaction score

1

(Created on Aug 30, 2023 by @iljakuklic) Mempool builds a transaction graph and assigns each transaction a score. The score is used to pick the best transactions to be included...