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

Warp Wallet integration and some fixes #36

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

bitcoincoltd
Copy link
Contributor

This pull request integrates warp wallet address generation ( https://keybase.io/warp/ ).

When generating new addresses users can choose the "Warp Wallet" option and enter a phrase/salt to generate a warp wallet address (which uses key stretching)

On the signing page if users enter a non-private key they will be prompted with options to convert their text into brainwallet key, or warpwallet key....so that users can sign directly with their brain wallet/warp wallet phrase.

Also includes a couple of fixes:

  • "Show" button next to password fields now toggles to "Hide" so that users can re-hide information once shown
  • Fixed mobile QR codes for transactions, they would too big for mobile.

bitcoincoltd added 7 commits December 21, 2015 17:36
Allow users to generate warp wallet address, and also use warp
wallet/brain wallet phrases to sign transactions
QR codes were fixed 512px on mobile making them too large, now resizes
on mobile.
Can now re-hide "shown" fields.
If private key on signing page is empty then hide the brainwallet
options.
# Conflicts:
#	index.html
#	js/coinbin.js
Resolve conflicts with master
@bitcoincoltd
Copy link
Contributor Author

I've updated this branch from the master so that it can be automatically merged.

@davidapple
Copy link

This is great. Would be good to see this integrated into Coinbin.

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.

None yet

2 participants