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

Going back from seed phrase import prevents log in #6740

Closed
danjm opened this issue Jun 21, 2019 · 0 comments · Fixed by #6758
Closed

Going back from seed phrase import prevents log in #6740

danjm opened this issue Jun 21, 2019 · 0 comments · Fixed by #6758
Assignees
Labels
area-UI Relating to the user interface. type-bug

Comments

@danjm
Copy link
Contributor

danjm commented Jun 21, 2019

Reproduction steps:

  • lock MetaMask
  • from the unlock screen, click to the import screen
  • click "Back"
  • enter your password and attempt to unlock
  • you will be taken to the import screen when you should be on the main screen
  • furthermore, refreshing or further trying to go back doesn't resolve the issue

Peek 2019-06-03 18-51

@danjm danjm added type-bug area-UI Relating to the user interface. labels Jun 21, 2019
@danjm danjm self-assigned this Jun 21, 2019
@Gudahtt Gudahtt self-assigned this Jun 25, 2019
Gudahtt added a commit to Gudahtt/metamask-extension that referenced this issue Jun 26, 2019
The back button on the import seed phrase page leaves the Redux store
with `appState.forgottenPassword` set to true, which prevents the user
from logging in. That flag is now unset when the user leaves the page.

Fixes MetaMask#6740
@whymarrh whymarrh added this to the UI Sprint 15 [June 24] milestone Jun 27, 2019
Gudahtt added a commit that referenced this issue Jun 27, 2019
The back button on the import seed phrase page leaves the Redux store
with `appState.forgottenPassword` set to true, which prevents the user
from logging in. That flag is now unset when the user leaves the page.

Fixes #6740
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-UI Relating to the user interface. type-bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants