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

Barclaycard Smartpay: Add app based 3ds auth and purchase #3327

Merged

Conversation

britth
Copy link
Contributor

@britth britth commented Sep 3, 2019

Like Adyen (#3298), we need to be able to create app based auth
and purchase requests, setting device_channel appropriately and
only sending notificationURL for browser based requests.

Remote:
36 tests, 87 assertions, 1 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
97.2222% passed

Unit:
28 tests, 143 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Unrelated pre-existing remote test failures:

  • test_successful_third_party_payout (invalid credentials)

@britth britth requested review from nfarve, jeremywrowe and a team September 3, 2019 19:02
Copy link

@nfarve nfarve left a comment

Choose a reason for hiding this comment

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

🐘

@britth britth force-pushed the barclaycard-app-based-3ds-updates branch from e3062c3 to 87936a3 Compare September 4, 2019 13:31
Like Adyen (activemerchant#3298), we need to be able to create app based auth
and purchase requests, setting device_channel appropriately and
only sending notificationURL for browser based requests.

Remote:
36 tests, 87 assertions, 1 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
97.2222% passed

Unit:
28 tests, 143 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Unrelated pre-existing remote test failures:
- test_successful_third_party_payout (invalid credentials)
@britth britth force-pushed the barclaycard-app-based-3ds-updates branch from 87936a3 to 19872bf Compare September 4, 2019 13:32
@britth britth merged commit 19872bf into activemerchant:master Sep 4, 2019
@britth britth deleted the barclaycard-app-based-3ds-updates branch September 4, 2019 15:32
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