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
Company Cards - The list of banks is not displayed when adding a company card #62246
Version Number: v9.1.46-5 Reproducible in staging?: Yes Reproducible in production?: No If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Email or phone of affected tester (no customers): Expensifail account Issue reported by: Applause Internal Team Device used: Mac 15.4.1/Safari, Desktop, iPhone 13 iOS 18.4.1/Safari, Hybrid App Component: Workspace Settings
👋 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.
@CortneyOfstad 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.
Please re-state the problem that we are trying to solve in this issue.
When adding a company card, after selecting the country and choosing the "Direct feed" connection type, the user is immediately redirected to the "Check your browser window" page. Instead, the user should see the "Who’s your bank?" page with a list of banks to choose from.
What is the root cause of that problem?
In the file src/pages/workspace/companyCards/addNew/SelectFeedType.tsx
after selecting "Direct feed", the flow transitions directly to the BANK_CONNECTION step, skipping the SELECT_BANK step.
As a result, the component responsible for displaying the list of banks (SelectBankStep) is never shown, and the user cannot manually select their bank.
Activity
melvin-bot commentedon May 17, 2025
Triggered auto assignment to @CortneyOfstad (
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 May 17, 2025
Triggered auto assignment to @MariaHCD (
DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.melvin-bot commentedon May 17, 2025
💬 A slack conversation has been started in #expensify-open-source
github-actions commentedon May 17, 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 May 17, 2025
@CortneyOfstad 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.
ganzz4 commentedon May 18, 2025
Proposal
Please re-state the problem that we are trying to solve in this issue.
When adding a company card, after selecting the country and choosing the "Direct feed" connection type, the user is immediately redirected to the "Check your browser window" page. Instead, the user should see the "Who’s your bank?" page with a list of banks to choose from.
What is the root cause of that problem?
In the file
src/pages/workspace/companyCards/addNew/SelectFeedType.tsx
after selecting "Direct feed", the flow transitions directly to the
BANK_CONNECTIONstep, skipping theSELECT_BANKstep.As a result, the component responsible for displaying the list of banks (
SelectBankStep) is never shown, and the user cannot manually select their bank.Problematic code snippet:
What changes do you think we should make in order to solve the problem?
In SelectFeedType.tsx, update the flow so that after selecting "Direct feed", the next step is
SELECT_BANKinstead ofBANK_CONNECTION.This will ensure the user is first shown the list of banks to choose from, and only after selecting a bank will the bank connection window open.
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
What alternative solutions did you explore? (Optional)
N/A
ganzz4 commentedon May 18, 2025
Demo
The video below demonstrates the bug fix in action.
Result.mp4
MariaHCD commentedon May 19, 2025
Not a deploy blocker - this feature is under beta and is currently a work in progress.
Without the beta, the navigation works correctly: https://expensify.slack.com/archives/C01GTK53T8Q/p1747642477414949?thread_ts=1747470921.807749&cid=C01GTK53T8Q
The correct navigation will be implemented here: #62117