-
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
Update Forte to the v3 api #3614
Comments
I gave that a try, but there are still some failures in the remote tests. Here's the branch: master...wsmoak:forte-api-v3 Note that there are some pre-existing failures on master in the echeck tests, see #3612 |
Update the Forte gateway to the new v3 API: https://restdocs.forte.net/?version=latest * In the endpoint URI, change v2 to v3 * In the endpoint URI, change /accounts to /organizations * In the X-Forte-Auth header change “Account” to “Organization” * Allow either organization_id or account_id to be used for configuration for backwards compatibility * Instead of act_XXXXXX use org_XXXXXX * remove the unused check_number attribute * change the 'disburse' action to 'credit' Fixes: activemerchant#3614 Unit: 20 tests, 67 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed Remote: 22 tests, 63 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
To provide a cleaner slate for the maintenance of the library, this PR/Issue is being labeled stale after 60 days without activity. It will be closed in 14 days unless you comment with an update regarding its applicability to the current build. Thank you! |
The Forte integration is currently using the older v2 api.
Here are the v3 docs: https://restdocs.forte.net/?version=latest
We have this list of tasks:
The text was updated successfully, but these errors were encountered: