Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Add Recover button to Accounts and warnings #5645

Merged
merged 4 commits into from May 18, 2017
Merged

Add Recover button to Accounts and warnings #5645

merged 4 commits into from May 18, 2017

Conversation

ngotchac
Copy link
Contributor

@ngotchac ngotchac commented May 17, 2017

Closes #5495

Add a new Recover button to the Accounts view, remove the restore from passphrase from account creation, and add warnings for empty phrases and phrases with less than 11 words.

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels May 17, 2017
defaultMessage='Key was created with Parity <1.4.5 on Windows'
id='createAccount.recoveryPhrase.warning.shortPhrase'
defaultMessage={ `The recovery phrase is less than 11 words.
This account have not been generated by Parity and might be insecure.
Copy link
Contributor

Choose a reason for hiding this comment

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

"has not"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

... Thanks ! ;)

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. B0-patch and removed A0-pleasereview 🤓 Pull request needs code review. labels May 17, 2017
@jacogr jacogr merged commit 8b1f0b7 into master May 18, 2017
@jacogr jacogr deleted the ng-recovery branch May 18, 2017 09:50
jacogr pushed a commit that referenced this pull request May 18, 2017
gavofyork pushed a commit that referenced this pull request May 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants