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

CLI wallet_import force wallet creation if requested #1357

Merged
merged 2 commits into from Nov 8, 2018

Conversation

argakiig
Copy link
Contributor

@argakiig argakiig commented Nov 7, 2018

When using the wallet_import currently the wallet_id must already exist on the node. With this change, passing --force 1 will allow it to create the wallet using the specified wallet_id and then import the accounts into that wallet from the supplied backup file

@argakiig argakiig added enhancement documentation This item indicates the need for or supplies updated or expanded documentation quality improvements This item indicates the need for or supplies changes that improve maintainability labels Nov 7, 2018
Copy link
Contributor

@SergiySW SergiySW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should allow backup-restore JSON with same wallet ID

@rkeene rkeene added this to the V17.0 milestone Nov 8, 2018
@rkeene rkeene merged commit d59d36f into nanocurrency:master Nov 8, 2018
@argakiig argakiig deleted the feature/wallet_import branch November 28, 2018 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This item indicates the need for or supplies updated or expanded documentation enhancement quality improvements This item indicates the need for or supplies changes that improve maintainability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants