Open izarutskaya opened 2 weeks ago
Triggered auto assignment to @lschurr (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.
Triggered auto assignment to @techievivek (DeployBlockerCash
), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.
:wave: 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:
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open Staging deploy checklist to see the list of PRs included in this release, then work quickly on the following:
Remember rule #2: Never un-assign yourself from a real DeployBlocker unless you are 100% sure someone else is assigned and will take care of it.
@lschurr 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.
We think this issue might be related to the #vip-split.
Going through deploy checklist to find offending PR
Looks this PR https://github.com/Expensify/App/pull/40785 Minor issue so removing deploy blocker label
I am Michał from Callstack - expert contributor group. I’d like to work on this job.
I'm also the author of the big PR that introduced this bug and have an idea on how to fix this - tabIndex is not being passed via props to children thus the focusing issue
PR is up (TS checks are failing due to unrelated errors in main). Ready to be reviewed. Hope it's ok that I've skipped proposal part - in the past it wasn't needed in a case when I was fixing regression from my previous PR
Rebased with latest main and all checks are green now
PR approved
Thanks!
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 1.4.77-11 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-06-06. :confetti_ball:
For reference, here are some details about the assignees on this 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:
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.74-1 Reproducible in staging?: Y Reproducible in production?: N Logs: https://stackoverflow.com/c/expensify/questions/4856 Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
When the split amount input is in focus, one press on Tab key will switch to the next input (production behavior).
Actual Result:
When the split amount input is in focus, one press on Tab key highlights the next user. Only the second press switches to the next input.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/599dae1a-50ca-42a9-85c2-b61b62cb73e1
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @lschurr