cygnet3 / danawallet

A mobile wallet for accepting bitcoin donations
MIT License
20 stars 4 forks source link

Improve load wallet error handling #49

Closed cygnet3 closed 2 months ago

cygnet3 commented 2 months ago

Right now the load wallet flow is not very future-proof. On startup, we try to read the wallet file. If we fail (either because no wallet exists, or we are unable to read the file), we take the user to the 'create wallet' screen. This effectively means that if we error for whatever reason, to a user, it will look like the wallet has been deleted.

We should improve this flow.

After reading the wallet file:

We should think more properly about how to deal with 'failing to load' state, but for now this is good enough.