-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
New Credit Card: Patagonia365 #5265
Conversation
24ba721
to
0d60f0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💳
0d60f0e
to
496f88f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, we need to make a change of the variable name to include an _
before 365
@@ -54,7 +54,8 @@ module CreditCardMethods | |||
'panal' => ->(num) { num&.size == 16 && in_bin_range?(num.slice(0, 6), PANAL_RANGES) }, | |||
'verve' => ->(num) { (16..19).cover?(num&.size) && in_bin_range?(num.slice(0, 6), VERVE_RANGES) }, | |||
'tuya' => ->(num) { num =~ /^588800\d{10}$/ }, | |||
'uatp' => ->(num) { num =~ /^(1175|1290)\d{11}$/ } | |||
'uatp' => ->(num) { num =~ /^(1175|1290)\d{11}$/ }, | |||
'patagonia365' => ->(num) { num =~ /^504656\d{10}$/ } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be patagonia_365
, sorry I didn't catch this sooner.
6065879
to
99f541b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet! LGTM!
* Include the new Patagonia365 card type * Remove the same bin number from the Maestro list * Include the Patagonia365 card in the supported card list for the following gateways: - Adyen - Checkout V2 - Cybersource - Cybersource Rest - D Local - Decidir - DecidirPlus - Global Collect - Mercado Pago - Redsys - Redsys Rest - Worldpay [SER-1414](https://spreedly.atlassian.net/browse/SER-1414) [SER-1415](https://spreedly.atlassian.net/browse/SER-1415) Unit Tests: ---------------- Finished in 48.473175 seconds. 6040 tests, 80425 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed Rubocop ---------------- 802 files inspected, no offenses detected
99f541b
to
b7ca8d3
Compare
Include the new Patagonia365 card type
Remove the same bin number from the Maestro list
Include the Patagonia365 card in the supported card list for the following gateways:
SER-1414
SER-1415
Unit Tests:
Finished in 48.473175 seconds.
6040 tests, 80425 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
Rubocop
801 files inspected, no offenses detected