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.

Sign up for GitHub

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

Jump to bottom

'InstallWizard' object has no attribute 'wallet_type' #168

Closed
PiRK opened this issue Dec 13, 2021 · 2 comments · Fixed by #171
Closed

'InstallWizard' object has no attribute 'wallet_type' #168

PiRK opened this issue Dec 13, 2021 · 2 comments · Fixed by #171

Comments

@PiRK
Copy link
Collaborator

PiRK commented Dec 13, 2021

This error was displayed in a dialog when trying to decrypt a wallet file with a hardware device (satochip, after typing in the pin code).
@PiRK
Copy link
Collaborator Author

PiRK commented Dec 13, 2021

The wallet file is probably a test wallet created while working on a development branch (#162 ?)

@PiRK
Copy link
Collaborator Author

PiRK commented Dec 13, 2021

The error is not related to the way the wallet was created. It happens when the first time the device is selected it returns another error (Failed to create a client for this device. Make sure it is in the correct state.), then I select the device a second time, and this time BaseWizard.on_device is called with the wrong purpose HWD_SETUP_NEW_WALLET instead of HWD_SETUP_DECRYPT_WALLET.

This was referenced Dec 13, 2021
Open
Merged
@PiRK PiRK closed this as completed in #171 Dec 13, 2021
PiRK added a commit that referenced this issue Dec 13, 2021
@PiRK
use the correct purpose when choosing hw device after setup_device fa…
efe9de2 
…iled

When for some reason the hw device fails setup (#170 for instance), make sure the hw selection is done again  with the same purpose. This prevents calling `on_device(purpose=HWD_SETUP_NEW_WALLET)` when the real purpose is to decrypt an existing wallet file (`HWD_SETUP_DECRYPT_WALLET`)
Closes #168
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

use the correct purpose when choosing hw device PiRK/ElectrumABC
1 participant
@PiRK