[HOLD for payment 2023-12-12] Web/mweb - Account settings - Assigned cards in Wallet do not display card logos

[lanitochka17]

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.7-0
Reproducible in staging?: Y
Reproducible in production?: N
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1. Navigate to staging.new.expensify.com
2. Log in with tester@applausecard.expensifail.com
3. Tap on settings icon in LHN
4. Select Wallet option
5. Observe blank CC logos seen for assigned cards

Expected Result:

Assigned cards in Wallet should display relevant card logos

Actual Result:

Assigned cards in Wallet do not display card logos

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• Windows: Chrome
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6298050_1701471167451.Recording__2262.mp4

View all open jobs on GitHub

[github-actions]

👋 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.

[melvin-bot]

Triggered auto assignment to @NikkiWines (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[lanitochka17]

Production:

[NikkiWines]

I see the same issue on web on staging, so not specific to mWeb. Also don't see the issue on prod. Checking dev now

[NikkiWines]

Same issue on dev. I think it's tied to #31776 and the changes made here

Checked out b7f71ca (the commit before the merge commit (56a368b) for #31776) and confirmed the icon loaded. Then checked out 56a368b and confirmed I could reproduce the bug 🐛

cc: @perunt @situchan @johnmlee101 @mountiny

[NikkiWines]

Looks like a pretty beneficial PR overall so I'd like to err against reverting it. Let's see if we can't resolve the issue in isolation

[NikkiWines]

Ah ok, looks like some of the BANK_NAMES defined here aren't quite right.

Some of the old bank names from here were checked with includes or startsWith instead of exact matches. As an example, expensify in the PR linked should be expensify card.

I'll check and see what bank names we use on the backend and spin up a PR later today to update the names accordingly

We have way too many options to match directly, so going to update this so we do a similar sort of matching that we did before