Skip to content
This repository has been archived by the owner. It is now read-only.

Reset password UX flow results in sign in on a 'sent email' screen #767

Closed
edwindotcom opened this issue Mar 19, 2014 · 6 comments
Closed

Reset password UX flow results in sign in on a 'sent email' screen #767

edwindotcom opened this issue Mar 19, 2014 · 6 comments
Labels

Comments

@edwindotcom
Copy link

@edwindotcom edwindotcom commented Mar 19, 2014

tracking in bugz also: https://bugzilla.mozilla.org/show_bug.cgi?id=985629

  1. in a signed out state or new profile on fx 29+, or in a state where you must 'reauthenticate'
  2. click hamburger menu > sign in or reconnect to sync
  3. click forgot password in sign in screen
  4. enter email of an fxa acct
  5. you should see a 'email sent screen'
  6. goto email, and click verify email link
  7. click hamburger menu > sign in or reconnect to sync

result: you see about:accounts but it's a email sent screen

expected: you should see sign in to FXA

@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Mar 19, 2014

@ryanfeeley's original UX: http://is.gd/FxA_Password_Reset

Auth server issue for supporting this: mozilla/fxa-auth-server#635

@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Mar 19, 2014

@pdehaan pdehaan added the ux label Mar 19, 2014
@pdehaan
Copy link
Contributor

@pdehaan pdehaan commented Mar 24, 2014

@ckarlof to try and repro.

@pdehaan pdehaan added this to the train-8 milestone Mar 24, 2014
@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Mar 24, 2014

Stop gap improvement in #793 provides a sign in link on the password reset "sent email" page. #787 tracks the work that will trigger about:accounts to transition back to the login screen. Longer term it would be nice that resetting the password in the same browser that triggered the reset would automatically log you back in.

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Mar 31, 2014

Does #832 fix the underlying problem here?

@nchapman nchapman modified the milestones: train-09 (Apr 7), train-08 (Mar 31) Mar 31, 2014
@pdehaan
Copy link
Contributor

@pdehaan pdehaan commented Apr 7, 2014

@pdehaan to try and repro after #832 fix.

@pdehaan pdehaan closed this Apr 7, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants