-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Hide Password mismatch incase both the passwords are empty #4607
Hide Password mismatch incase both the passwords are empty #4607
Conversation
@mananjadhav You can use the Fixes keyword to close that issue. Fixes #4606 |
@parasharrajat Okay. Is it that for fresh tasks/issues we don't use Fixes and for regression we do? I am told adding Fixes closes the issue when PR is merged. |
No, it's not like that. It is just that there is nothing left to do with that issue after you have merged the fix. So it saves some time. Also, it will create a link between the issue and PR. So anyone can find what fixed it. |
Thanks for the tip. |
@AndrewGable Can you please review this and probably merge too? #4665 is blocked by this. |
Trying to untangle the duplicates before merging this. |
|
CP-ing to resolve deploy blocker. |
…tion Hide Password mismatch incase both the passwords are empty (cherry picked from commit 10a422a)
🚀 Cherry-picked to staging by @AndrewGable in version: 1.0.85-7 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
🚀 Deployed to staging by @AndrewGable in version: 1.0.85-10 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀
|
This has been deployed to production and is now subject to a 7-day regression period. |
@alex-mechler Can you please help review this?
Details
Show Password mismatch error only if New Password is entered.
Regression from #4367
Fixed Issues
$ #4606
$ #4665
Tests
Apart from the test cases mentioned here #4409
QA Steps
Tested On
Screenshots
Web
change-password-val-web.mp4
Mobile Web
change-password-val-mweb.mp4
Desktop
change-password-val-desktop.mp4
iOS
change-password-val-ios.mp4
Android
change-password-val-android.mp4