Skip to content
This repository has been archived by the owner on Jan 20, 2024. It is now read-only.

Support Unionpay #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

limsangjin12
Copy link

The PR should summarize what was changed and why. Here are some questions to
help you if you're not sure:

  • What behavior was changed?
    Added Unionpay card type.

Checklist - While not every PR needs it, new features should consider this list:

  • Does this have tests?
  • Does this have documentation?
  • Does this break the public API (Requires major version bump)?
  • Is this a new feature (Requires minor version bump)?

@maxkramer
Copy link
Owner

maxkramer commented May 24, 2019

Hi @limsangjin12 thanks for the PR, sorry I didn't see this sooner. I can't find much in terms of validation methods for this kind of card other than it starting with 62 and being 16-19 digits long, which doesn't match the regex that you supplied. Please could you confirm?

Ref: https://www.getnewidentity.com/unionpay-credit-card.php
https://www.six-payment-services.com/dam/download/datasheets/110004302_DS_SecurityFeatures_UnionPay_INT_EN_opt.pdf

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants