Skip to content

[EP-2614] Show credit card type icon#1258

Merged
dr-bizz merged 2 commits intomasterfrom
2614-credit-card-type
Jan 22, 2026
Merged

[EP-2614] Show credit card type icon#1258
dr-bizz merged 2 commits intomasterfrom
2614-credit-card-type

Conversation

@canac
Copy link
Contributor

@canac canac commented Nov 17, 2025

Description

Show the credit card type icon next to the credit card number input after the user starts typing it in.

I made the icon small because it can make the input too narrow to show the entire number in normal checkout. I didn't want to make the "Name on Card" input any narrower because it is already a little small. Let me know if you have any feedback on layout, sizing, or placement.

EP-2614

Testing

Screenshot 2025-11-17 at 12 00 23 PM Screenshot 2025-11-17 at 12 00 43 PM

Checklist:

  • I have given my PR a title with the format "EP-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels (Add the label "On Staging" to get the branch automatically merged into staging)
  • I have requested a review from another person on the project
  • I have checked that stage-branch-merger successfully merged my branch to staging or manually merged it myself
  • I have tested my changes in staging for regular checkout
  • I have tested my changes in staging for branded checkout

@canac canac requested a review from dr-bizz November 17, 2025 18:02
@canac canac self-assigned this Nov 17, 2025
@canac canac added the On Staging Will be merged into the staging branch by GitHub Actions label Nov 17, 2025
@stage-branch-merger
Copy link

I see you added the "On Staging" label, I'll get this merged to the staging branch!

@dr-bizz
Copy link
Contributor

dr-bizz commented Nov 20, 2025

With testing this, I realised we don't accept Diners Club or UnionPay. Which might not be a big issue.

https://docs.stripe.com/testing

@wrandall22
Copy link
Contributor

We at least used to accept Diners Club. I forget if we stopped at some point for some reason.

Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this. We will not be able to deploy this until the new year, due to the code freeze. I will approve when freeze is over.

What are your thoughts on having the card icon in the input?

Image

@canac
Copy link
Contributor Author

canac commented Nov 20, 2025

Diner's Club should work. There's a spec for it. UnionPay is not supported by the type detector in cru-payments.

What are your thoughts on having the card icon in the input?

I like that idea! I'll probably circle back to that after our MPDX work. It shouldn't be too hard to implement.

@dr-bizz dr-bizz force-pushed the 2614-credit-card-type branch from 2ff8214 to 9a7c07e Compare January 22, 2026 18:29
@dr-bizz dr-bizz self-requested a review January 22, 2026 18:32
Copy link
Contributor

@dr-bizz dr-bizz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA done. I will create a new PR for the secure change as that is not part of this PR

@dr-bizz dr-bizz merged commit 8423f78 into master Jan 22, 2026
5 checks passed
@dr-bizz dr-bizz deleted the 2614-credit-card-type branch January 22, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

On Staging Will be merged into the staging branch by GitHub Actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants