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
client/db: add account backup and restore. #210
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is almost exactly what I had in mind, but I do want to standardize this format at the level of the client/db package. So probably a type dexdb.AccountBackup
which has a Serialize() []byte
method. This ensures that the backup can be imported into a different database backend in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small tweak for me. @buck54321 might have other suggestions.
Also, please try not to squash commits during review. Even after the PR is approved it's not necessary to squash since I just do it. Review is way easier if we see all the incremental diffs.
This adds AccountBackup type for backup and restore functionality for user accounts. RandomWallet has also been updated to use the current time for balance update times to fix an issue with TestWallets.
This adds
AccountBackup
type for backup and restore functionality for user accounts.RandomWallet
has also been updated to use the current time for balance update times to fix an issue withTestWallets
.Resolves #179.