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

Forgot password - Validate Code is likely expired error not handled #5076

Closed
isagoico opened this issue Sep 3, 2021 · 9 comments
Closed
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Weekly KSv2

Comments

@isagoico
Copy link

isagoico commented Sep 3, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Launch the app and enter email
  2. In password page tap forgot password
  3. Tap the magic link and setup password changing the URL to include staging
  4. Setup password and logout
  5. Tap the same magic link to setup the password
  6. Enter a password and hit submit

Expected Result:

Error message should be shown to the user, similar to expensify.com
image

Actual Result:

Magic link can be reused and after entering new password and confirm password, tapping setup password triggers new magic link. Password is not setup though

Workaround:

Unknown

The ideal solution here would be to handle the error that is thrown after submitting the form the 2nd time and show the error message above:
image

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.0.93-0

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5222384_Forgot_Password-1.mp4

Expensify/Expensify Issue URL:

View all open jobs on GitHub

@MelvinBot
Copy link

Triggered auto assignment to @Jag96 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@MelvinBot
Copy link

@Jag96 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@Jag96
Copy link
Contributor

Jag96 commented Sep 7, 2021

When testing on staging and production, I can change my password successfully, I sign out, and I click the link again which opens the same page. If I try submitting the form with a new password I get a console error:

image

It doesn't look like this case is handled, so I agree this should be fixed. On expensify.com we have the same kind of error, the error message doesn't come up until the user submits the form again so I think that part is fine.

Making external, all we should be doing here is displaying an error when we get the json result above. The error message can match what we have on expensify.com:
image

@MelvinBot MelvinBot removed the Overdue label Sep 7, 2021
@Jag96 Jag96 added the External Added to denote the issue can be worked on by a contributor label Sep 7, 2021
@MelvinBot
Copy link

Triggered auto assignment to @jliexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@jliexpensify
Copy link
Contributor

@Jag96 - just clarifying: should we be editing @isagoico's original issue to reflect your proposal to make it clearer in regards to what we want?

Essentially - replacing the "Expected Result" in the original issue with your JSON screenshot, and error message screenshot?

@Jag96 Jag96 changed the title Forgot password - User can use forgot password magic link more than once Forgot password - Validate Code is likely expired error not handled Sep 8, 2021
@Jag96
Copy link
Contributor

Jag96 commented Sep 8, 2021

@jliexpensify good call, updated!

@jliexpensify
Copy link
Contributor

@MelvinBot MelvinBot added Weekly KSv2 Help Wanted Apply this label when an issue is open to proposals by contributors and removed Daily KSv2 labels Sep 8, 2021
@akshayasalvi
Copy link
Contributor

akshayasalvi commented Sep 8, 2021

Is this duplicate of #4737

I have a PR in progress for the linked issue and I am guessing it’s the same behavior

@Jag96
Copy link
Contributor

Jag96 commented Sep 8, 2021

Ah, good catch @akshayasalvi! Yes, I think we can close this one then since it's a dupe. @jliexpensify can you please close out the Upwork job?

@Jag96 Jag96 closed this as completed Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants