You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[Due for payment 2025-06-11] [$250] Onboarding - Each time user reenters a digit in magic code field, it validates #60818
Version Number: 9.1.32-0 Reproducible in staging?: Yes Reproducible in production?: Unable to check - No OTP prompt on prod If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only If this was caught during regression testing, add the test name, ID and link from TestRail:#54912 Email or phone of affected tester (no customers):letos9a575@agiuse.com Issue reported by: Applause Internal Team Device used: Mac 15.3 / Chrome App Component: Other
Action Performed:
Go to staging.new.expensify.com
Log in with this private domain email - @agiuse.com
Enter name > Continue.
On magic code page, enter any code (any code will be the incorrect code).
After app throws error, click on the first digit field.
Enter any digit.
Expected Result:
After changing the first digit, the code input field will move to the next digit.
Actual Result:
After changing the first digit, the code input field does not move to the next digit. The code revalidates each time a digit is changed. If user wants to change the entire fields, they will have to click on the field six times to reenter the code.
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
Identify the pull request that introduced this issue and revert it.
@greg-schroeder FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.
Hm... I am not sure what the expected behaviour here is. @rlinoz We did not add a submit button in the UI, and as such, we have to submit when the field has all characters filled in.
changed the title [-][$250] Onboarding - Each time user reenters a digit in magic code field, it validates[/-][+][Due for payment 2025-06-11] [$250] Onboarding - Each time user reenters a digit in magic code field, it validates[/+]on Jun 4, 2025
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.58-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:
@parasharrajat / @allroundexperts@greg-schroeder@parasharrajat / @allroundexperts The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]
1a. Result of the original design (eg. a case wasn't considered)
1b. Mistake during implementation
1c. Backend bug
1z. Other:
Where bug was reported:
2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
2b. Reported on staging (eg. found during regression or PR testing)
2d. Reported on a PR
2z. Other:
Who reported the bug:
3a. Expensify user
3b. Expensify employee
3c. Contributor
3d. QA
3z. Other:
[Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: Not an issue. We changed the design.
[Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: Not needed
[Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.
[BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:
Regression Test Proposal
Test:
Log in with this private domain email - e.g. @agiuse.com
Enter name > Continue.
On magic code page, enter any code (any code will be the incorrect code).
After app throws error, click on the first digit field.
Enter any digit.
Verify that after changing the first digit, the code input field will move to the next digit.
Payment summary here looks good and regression test is filed. Both @parasharrajat and @allroundexperts can make a manual request via ND for payment tomorrow, 2025-06-11.
Activity
melvin-bot commentedon Apr 24, 2025
Triggered auto assignment to @greg-schroeder (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.melvin-bot commentedon Apr 24, 2025
Triggered auto assignment to @rlinoz (
DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.melvin-bot commentedon Apr 24, 2025
💬 A slack conversation has been started in #expensify-open-source
github-actions commentedon Apr 24, 2025
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
nlemma commentedon Apr 24, 2025
@greg-schroeder FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.
rlinoz commentedon Apr 24, 2025
Probably from #54912 that introduced the feature?
cc: @parasharrajat @allroundexperts
allroundexperts commentedon Apr 24, 2025
Hm... I am not sure what the expected behaviour here is. @rlinoz We did not add a submit button in the UI, and as such, we have to submit when the field has all characters filled in.
49 remaining items
[-][$250] Onboarding - Each time user reenters a digit in magic code field, it validates[/-][+][Due for payment 2025-06-11] [$250] Onboarding - Each time user reenters a digit in magic code field, it validates[/+]melvin-bot commentedon Jun 4, 2025
Reviewinglabel has been removed, please complete the "BugZero Checklist".melvin-bot commentedon Jun 4, 2025
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.58-4 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 2025-06-11. 🎊
For reference, here are some details about the assignees on this issue:
melvin-bot commentedon Jun 4, 2025
@parasharrajat / @allroundexperts @greg-schroeder @parasharrajat / @allroundexperts The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]
parasharrajat commentedon Jun 9, 2025
BugZero Checklist:
Bug classification
Source of bug:
Where bug was reported:
Who reported the bug:
[Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: Not an issue. We changed the design.
[Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: Not needed
[Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.
[BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:
Regression Test Proposal
Test:
Verify that after changing the first digit, the code input field will move to the next digit.
Do we agree 👍 or 👎
greg-schroeder commentedon Jun 10, 2025
Payment summary here looks good and regression test is filed. Both @parasharrajat and @allroundexperts can make a manual request via ND for payment tomorrow, 2025-06-11.
melvin-bot commentedon Jun 11, 2025
Payment Summary
Upwork Job
BugZero Checklist (@greg-schroeder)
parasharrajat commentedon Jun 27, 2025
Payment requested as per #60818 (comment)
JmillsExpensify commentedon Jul 9, 2025
$250 approved for @parasharrajat
trjExpensify commentedon Jan 5, 2026
approved $250 for @allroundexperts.