Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add version numbers to the backups and implement an import migration path for older backups #63

Open
jojobyte opened this issue Mar 15, 2024 · 0 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@jojobyte
Copy link
Contributor

jojobyte commented Mar 15, 2024

Error

I get this error after loading a wallet backup:

utils.js:1062 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '0')
    at getAddressIndexFromUsage (utils.js:1062:32)
    at getAccountWallet (wallet.js:1144:29)
    at async main (main.js:622:14)
getAddressIndexFromUsage	@	utils.js:1062
getAccountWallet	@	wallet.js:1144
await in getAccountWallet (async)		
(anonymous)	@	main.js:1150

Image

As you can see, my screen is blank, so I can't test yet.

Originally posted by @riongull in #41

Diagnosis

This appears to be related to using an older wallet backup that was created before we added change address support.

We likely need to add version numbers to the backups and have some form of import migration path for older backups.

@jojobyte jojobyte added bug Something isn't working enhancement New feature or request labels Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

1 participant