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

Bip38 import #816

Closed
wants to merge 9 commits into from
Closed

Bip38 import #816

wants to merge 9 commits into from

Conversation

wozz
Copy link
Contributor

@wozz wozz commented Sep 4, 2014

Bip38 import capability

@ecdsa
Copy link
Member

ecdsa commented Sep 14, 2014

travis fails because decr is not assigned in bitcoin.py line 83.
in terms of UX, I think we should we keep a single sweep menu, and detect if the key is encrypted.

@wozz
Copy link
Contributor Author

wozz commented Sep 14, 2014

The reason I didn't do the same sweep menu is that if there are multiple private keys, the password dialog could get confusing. It would need to ask for the password to each key individually. I can change it around to do that instead of a separate menu option.

…p38_import

Conflicts:
	.travis.yml
	gui/qt/main_window.py
	setup.py
@ulrichard
Copy link
Contributor

My vote for this pull request.
In addition, zbarcam would be nice.

@ecdsa
Copy link
Member

ecdsa commented Feb 11, 2015

please resubmit as a plugin; this feature requires non-python dependencies.

@ecdsa ecdsa closed this Feb 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants