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

COLD-370 Add unionpay as an accepted card type #1224

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

KyleG43
Copy link
Contributor

@KyleG43 KyleG43 commented Mar 6, 2024

This PR adds UnionPay to the list of accepted card types and assigns a Font Awesome icon to it.

Copy link

kermitapp bot commented Mar 6, 2024

@KyleG43 KyleG43 force-pushed the COLD-365-UnionPay-Spike branch 3 times, most recently from 444b230 to 4888106 Compare March 8, 2024 20:05
@@ -33,6 +40,7 @@ const ICONS: {
'master-card': 'cc-mastercard',
discover: 'cc-discover',
jcb: 'cc-jcb',
unionpay: 'credit-card',
Copy link
Contributor Author

@KyleG43 KyleG43 Mar 8, 2024

Choose a reason for hiding this comment

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

I couldn't find a Font Awesome icon for UnionPay, so I used the default credit card icon. We may want to use the discover Icon instead

@KyleG43 KyleG43 changed the title Add unionpay as an accepted card type COLD-370 Add unionpay as an accepted card type Mar 8, 2024
Copy link

kermitapp bot commented Mar 8, 2024

Copy link

@sethvanb sethvanb left a comment

Choose a reason for hiding this comment

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

LGTM

@KyleG43 KyleG43 force-pushed the COLD-365-UnionPay-Spike branch from 4888106 to 8582543 Compare March 8, 2024 22:28
Copy link
Contributor

github-actions bot commented Mar 8, 2024

Released prerelease version 8.10.4-COLD-365-UnionPay-Spike-669efdd.0.
You may now run npm install @appfolio/react-gears@COLD-365-UnionPay-Spike

@KyleG43 KyleG43 merged commit 0b890ce into master Mar 8, 2024
4 checks passed
@KyleG43 KyleG43 deleted the COLD-365-UnionPay-Spike branch March 8, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants