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

Wallet - Account number field is not focused #54742

Closed
4 of 8 tasks
IuliiaHerets opened this issue Jan 2, 2025 · 15 comments
Closed
4 of 8 tasks

Wallet - Account number field is not focused #54742

IuliiaHerets opened this issue Jan 2, 2025 · 15 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Reviewing Has a PR in review

Comments

@IuliiaHerets
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: v9.0.80-2
Reproducible in staging?: Yes
Reproducible in production?: N/A - new feature, doesn't exist in prod
If this was caught during regression testing, add the test name, ID and link from TestRail: #52322
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 14.5/ Chrome
App Component: User Settings

Action Performed:

  1. Login to NewDot.
  2. Go to Settings > Wallet > Add bank account.
  3. Click Add bank account.
  4. Select any county other than United States(or select India)
  5. Click Next.

Expected Result:

Account number field is focused.

Actual Result:

Account number field is not focused.

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6704918_1735781701545.Screen_Recording_2025-01-02_at_4.18.11_at_night.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added DeployBlockerCash This issue or pull request should block deployment Bug Something is broken. Auto assigns a BugZero manager. labels Jan 2, 2025
Copy link

melvin-bot bot commented Jan 2, 2025

Triggered auto assignment to @JmillsExpensify (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.

Copy link

melvin-bot bot commented Jan 2, 2025

Triggered auto assignment to @techievivek (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@melvin-bot melvin-bot bot added the Daily KSv2 label Jan 2, 2025
Copy link

melvin-bot bot commented Jan 2, 2025

💬 A slack conversation has been started in #expensify-open-source

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Jan 2, 2025
Copy link
Contributor

github-actions bot commented Jan 2, 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:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@Beamanator
Copy link
Contributor

Also don't believe this should be a blocker

@Beamanator Beamanator added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Jan 2, 2025
@melvin-bot melvin-bot bot added the Overdue label Jan 6, 2025
@techievivek techievivek added the Reviewing Has a PR in review label Jan 6, 2025
@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2025
@techievivek
Copy link
Contributor

This is in review, #54798 should fix this.

@DylanDylann
Copy link
Contributor

@techievivek #54798 will not fix this problem

@shubham1206agra @MonilBhavsar @joekaufmanexpensify Let's discuss this problem separately here

@DylanDylann
Copy link
Contributor

DylanDylann commented Jan 10, 2025

Note: We have the same bug on the below screens

Screenshot 2025-01-10 at 16 08 12 Screenshot 2025-01-10 at 16 11 55

Copy link

melvin-bot bot commented Jan 13, 2025

@JmillsExpensify, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Jan 15, 2025

@JmillsExpensify, @techievivek Huh... This is 4 days overdue. Who can take care of this?

@techievivek
Copy link
Contributor

@MonilBhavsar @shubham1206agra can you please take a look at the comment here #54742 (comment)

@DylanDylann
Copy link
Contributor

Coming from #54798 (comment)

#54742 - @MonilBhavsar I don't think this is an issue since we don't focus any input if page contains more than one input + it would be hard to focus in case of dynamically rendered options.

@joekaufmanexpensify @MonilBhavsar Do you agree with this?

Copy link

melvin-bot bot commented Jan 23, 2025

@JmillsExpensify, @techievivek Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Copy link

melvin-bot bot commented Jan 27, 2025

@JmillsExpensify, @techievivek Still overdue 6 days?! Let's take care of this!

@MonilBhavsar
Copy link
Contributor

@joekaufmanexpensify @MonilBhavsar Do you agree with this?

Sorry for missing this. I see this pattern in the App and agree with this. Feel free to reopen if required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

6 participants