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

Seed Phrase Recovery #1394

Closed
wants to merge 34 commits into from
Closed

Seed Phrase Recovery #1394

wants to merge 34 commits into from

Conversation

VolkerSchiewe
Copy link

@VolkerSchiewe VolkerSchiewe commented Jul 30, 2019

Open TODOs:

  • Design for input seed phrase
  • Snapshot test for input seed phrase screen
  • Add iOS back button on SeedPhraseComponent
  • Show waring only after the user received external credentials

New Features

  • Initial selection of new Identity or recovery
  • Move seed phrase creation to settings
  • repeat seed phrase recovery screen

VolkerSchiewe and others added 19 commits July 24, 2019 10:49
# Conflicts:
#	src/routes.ts
#	src/ui/recovery/container/seedPhrase.tsx
#	src/ui/settings/containers/settings.tsx
JolocomTheme should be imported from the generic theme file, as the
font name is different on iOS and Android.
We are using position relative and absolute to put the mnemonic phrase
in the middle of the screen, using the length of the current word with
an average character length.
There may be a way to calculate the actual rendered length of the word
and use that to position more accurately.
…rase

Fix/recovery/center mnemonic phrase
# Conflicts:
#	src/actions/registration/index.ts
#	src/routes.ts
#	src/ui/landing/containers/landing.tsx
#	src/ui/registration/containers/seedPhrase.tsx
#	src/ui/settings/containers/settings.tsx
#	tests/actions/registration/__snapshots__/index.test.ts.snap
#	tests/ui/containers/__snapshots__/seedPhrase.test.tsx.snap
#	tests/ui/containers/landing.test.tsx
package.json Outdated
@@ -55,7 +55,7 @@
"yarn": "^1.16.0"
},
"scripts": {
"start": "node node_modules/react-native/local-cli/cli.js start",
"start": "adb reverse tcp:8081 tcp:8081 && node node_modules/react-native/local-cli/cli.js start",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this? metro takes care of this automatically

Copy link
Author

Choose a reason for hiding this comment

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

Does it? it only worked for me with run:android but when I just use start I got a Could not connect to development server error...

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah I've had to run adb reverse almost every time

@VolkerSchiewe
Copy link
Author

closing in favor of #1418 and #1419 and 1266/seedphrase-input

@VolkerSchiewe VolkerSchiewe deleted the 1266/recovery branch September 9, 2019 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants