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

Decidir: Pass CVV for NT #5205

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Decidir: Pass CVV for NT #5205

merged 1 commit into from
Aug 14, 2024

Conversation

almalee24
Copy link

This changes will allow CVV, card number, exp month and exp year.

Remote
27 tests, 97 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed

@almalee24 almalee24 requested a review from a team August 6, 2024 21:41
@aenand
Copy link
Contributor

aenand commented Aug 7, 2024

Decidir wants the Network Token data and the raw pan? What's the point of sending a NT then

@almalee24
Copy link
Author

Decidir wants the Network Token data and the raw pan? What's the point of sending a NT then

It's not Decidir but a Visa mandate that we need to send CVV for NT transactions. This is meant to just send CVV but I was think we might as well just send everything.

@almalee24 almalee24 force-pushed the decidir_nt_cvv branch 2 times, most recently from 0512cd6 to 60df77e Compare August 9, 2024 18:13
Copy link
Contributor

@bradbroge bradbroge left a comment

Choose a reason for hiding this comment

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

LGTM

This changes will allow CVV to be passed for NTs.

Remote
27 tests, 97 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@almalee24 almalee24 merged commit edf22c3 into master Aug 14, 2024
5 checks passed
@almalee24 almalee24 deleted the decidir_nt_cvv branch August 14, 2024 22:15
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.

3 participants