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

Wrong Validation message is shown on change password page. #4606

Closed
parasharrajat opened this issue Aug 12, 2021 · 9 comments
Closed

Wrong Validation message is shown on change password page. #4606

parasharrajat opened this issue Aug 12, 2021 · 9 comments
Assignees
Labels
Daily KSv2

Comments

@parasharrajat
Copy link
Member

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Open Change Password Page on newDot Web.
  2. Click on the New Password field. Do not type anyting.
  3. Click on Confirm Password Field. Do not type anyting.

Expected Result:

No validation error should be shown

Actual Result:

Validation error Password must match is shown.

Workaround:

None. UI issue.

Platform:

Where is this issue occurring?

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

Version Number: 1.0.84.0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
image

Expensify/Expensify Issue URL:

View all open jobs on Upwork

@parasharrajat parasharrajat added AutoAssignerTriage Auto assign issues for triage to an available triage team member Daily KSv2 labels Aug 12, 2021
@MelvinBot
Copy link

Triggered auto assignment to @michaelhaxhiu (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Aug 12, 2021
@mananjadhav
Copy link
Collaborator

Looks like a regression from my PR #4367. On it.

@mananjadhav
Copy link
Collaborator

PR Raised.

@parasharrajat
Copy link
Member Author

Cool, please close this issue via that PR.

@AndrewGable
Copy link
Contributor

PR merged

@MelvinBot
Copy link

@michaelhaxhiu Huh... This is 4 days overdue. Who can take care of this?

@mananjadhav
Copy link
Collaborator

@michaelhaxhiu we can close this issue as the fix was merged and deployed

@MelvinBot
Copy link

@michaelhaxhiu Still overdue 6 days?! Let's take care of this!

@parasharrajat
Copy link
Member Author

The issue is resolved and fixed here #4607

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2
Projects
None yet
Development

No branches or pull requests

5 participants