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

Backup phrase bug #394

Merged
merged 2 commits into from
May 8, 2018
Merged

Backup phrase bug #394

merged 2 commits into from
May 8, 2018

Conversation

schnogz
Copy link
Contributor

@schnogz schnogz commented May 8, 2018

Description

Fixes an issue where the 12th word, if randomly selected, would never draw a text input box.

Change Type

Bug Fix

Testing Steps

Go to recovery phrase in settings and keep looping through the wizard until the 12th word is selected for verification in the final verifcation step.

PR Creator Checklist

  • Code compiles correctly
  • No lint issues exist (verified via yarn lint)
  • All unit tests pass (verified via yarn test)
  • Updated README.md and other documentation (if necessary)

PR Reviewer Checklist

  • Change validated and application spot checked
  • Code styles and best practices met
  • Code is readable and commented when necessary

@plondon
Copy link
Contributor

plondon commented May 8, 2018

Nice catch

@plondon plondon merged commit dc7568c into development May 8, 2018
@plondon plondon deleted the feature/backup-phrase-bug branch May 8, 2018 23:03
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.

2 participants