-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[PAID] [HOLD for payment 2023-02-14] [$1000] Login - The line is green when incorrect password entered #14622
Comments
Job added to Upwork: https://www.upwork.com/jobs/~017759eff2106d5a57 |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @aimane-chnaif ( |
Bug0 Triage ChecklistNote: see this SO for more information.
|
I couldn't recreate this and realized I didn't have NewDot app on my android on staging. I'm working on getting it downloaded and then I'll finish testing. |
I have a draft PR up for this and should work some more on it today! |
@luacmartins I assume you were able to recreate this okay then right? I am struggling to get newdot staging app on my phone so I was going to ask in Slack for someone to test. It's not recreatable on prod for me. If you can recreate and feel it's a legit bug I will take your testing as confirmation and move this forward. |
@strepanier03 yes, I can reproduce this and I'm working on a fix for it. What issues are you having with installing the staging App? |
Thanks for your help in Echat @luacmartins - I'm all set on staging now and can recreate this as well. |
|
Upwork job price has been updated to $1000 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.66-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-02-14. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Updated the payment checklist and am working on the reg test step now. |
@aimane-chnaif would you mind working on the BZ checklist? |
I don't think any PR caused this regression. This bug existed from the beginning when sign in pages are implemented. |
Regression Test Proposal
|
Oooh, I didn't realize that the regression proposal changed to the C+ now and wasn't part of my steps in the checklist. The way our regression tests are written we'd need to add tests for every input or update every test that has an input step. I'm not 100% sure what the best practice is there. I'll wait for @luacmartins to weigh in but I'm thinking our options are:
|
IIRC the way we handle these is to add a description under the |
@luacmartins - Sounds good. How do I know which field inputs have validation on them so I can narrow down which tests we need to update? |
I think that we should include the pages below, because they don't use our Form component but should behave similarly. Other inputs are handled by the Form component and I don't think we need to worry about those here.
|
Thank you @luacmartins - I'll work on the GH for this tomorrow. I've also updated the checklist with Aimane's comment regarding the PR that intro'd the bug and will be ready to pay out Upwork tomorrow. |
@aimane-chnaif - I've sent an invite to the job directly to you, once you accept I'll send the offer and check back in this afternoon to see if I can pay it out. |
Just sent the offer @aimane-chnaif - Before I leave today I'll check again and if it's accepted I'll pay out, otherwise I'll check again tomorrow. |
GH for the reg test is here. I'm still discussing some things on the reg test buddy check but I made the GH so this GH could be closed. Thanks again @aimane-chnaif! |
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:
Expected Result:
The line should be red or no focus when an invalid password is entered
Actual Result:
The line is green when incorrect password entered
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.60.0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: