-
Notifications
You must be signed in to change notification settings - Fork 5
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
create master key & import master key #29
Conversation
chescalante
commented
Oct 22, 2021
•
edited
edited
0174596
to
0d3f55c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I merged from main
and tests started failing. There was no big change... I guess something is wrong in the tests, it never ran locally.
Only one change requested: where to save the mnemonic. Great job! This change should also modify how tests are built. I didn't revert my changes.
useEffect(() => { | ||
saveMnemonic(mnemonic) | ||
}, [mnemonic]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would call this in Confirm
and Import
components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging with describe.skip
for the tests. Opening a new issue.