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

Mobile FTUE: Password recovery (meta) #135

Closed
Tracked by #5151 ...
daniellekirkwood opened this issue Jan 26, 2022 · 10 comments
Closed
Tracked by #5151 ...

Mobile FTUE: Password recovery (meta) #135

daniellekirkwood opened this issue Jan 26, 2022 · 10 comments
Assignees
Labels
A-Onboarding The users first experience of Element, whether that be first impressions or first tasks. T-Epic Issue is at Epic level Z-FTUE

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Jan 26, 2022

When the user has forgotten their password, we need to ensure we can get them back on track

Figma: https://www.figma.com/file/xdV4PuI3DlzA1EiBvbrggz/Login-Flow-v2?node-id=2523%3A42803

Screenshot 2022-02-21 at 18 43 26

  • Add a "sign out of all sessions" option. Default unticked.
@daniellekirkwood
Copy link
Contributor Author

@daniellekirkwood daniellekirkwood changed the title Mobile FTUE: Password recovery flow Mobile FTUE: Password recovery (meta) Feb 21, 2022
@daniellekirkwood daniellekirkwood added A-Onboarding The users first experience of Element, whether that be first impressions or first tasks. Z-FTUE labels Mar 7, 2022
@hughns
Copy link
Member

hughns commented Apr 8, 2022

@daniellekirkwood can I suggest that the "sign out all devices" option from matrix-org/matrix-react-sdk#8259 is incorporated into these mobile designs?

@daniellekirkwood
Copy link
Contributor Author

@hughns Happy to discuss that for sure! Is this the reset password flow when the user is signed in, or when the user is attempting to sign in?

@hughns
Copy link
Member

hughns commented Apr 11, 2022

@daniellekirkwood the linked issue covers both cases, but I am only suggesting incorporating a "Sign out all devices" checkbox similar to the following on the forgot password part of signing in:

image

@daniellekirkwood
Copy link
Contributor Author

@amshakal & @ouchadam & @pixlwave
Is this do-able for us? How long would this scope change set us back? WDYT?
Thanks!

@ouchadam
Copy link

ouchadam commented Apr 11, 2022

very doable https://spec.matrix.org/v1.2/client-server-api/#post_matrixclientv3accountpassword would lean towards this being a small/small-medium amount of work

it seems like something that could be done in a second iteration though (we'll probably want the extra behaviour in the settings -> change password flow as well)

@hughns
Copy link
Member

hughns commented May 3, 2022

Re: scope. Please note that there is need to vary behaviour based on which version of the matrix spec the homeserver supports. The logic is captured in matrix-org/matrix-react-sdk#8259

@ara4n
Copy link
Member

ara4n commented May 5, 2022

Real-life microcosm of why Hugh's refinement is important:

image

@novocaine novocaine added the T-Epic Issue is at Epic level label May 6, 2022
@daniellekirkwood
Copy link
Contributor Author

@hughns We discussed this yesterday as a team and have added "sign out of all sessions" to the definition of done for this issue. I'm sure @ouchadam and @pixlwave will reach out with any questions if we have them :)

@jdauphant
Copy link

jdauphant commented Aug 4, 2022

Can we update it for the web version ? (having the same new workflow as mobile)

cc @gaelledel @giomfo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Onboarding The users first experience of Element, whether that be first impressions or first tasks. T-Epic Issue is at Epic level Z-FTUE
Projects
None yet
Development

No branches or pull requests

6 participants