-
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
PaymentExpress: correct endpoints #4827
Conversation
@steveh do you have more context on the old endpoints not working? I do know of some merchants still using the old endpoints successfully. |
@aenand I should have been more specific, only UAT is not working so far. We run integration tests on build against the UAT endpoint. On around the 4th of July it was removed: $ nslookup uat.paymentexpress.com
** server can't find uat.paymentexpress.com: NXDOMAIN Shown in history too: https://dnshistory.org/historical-dns-records/a/uat.paymentexpress.com |
Ah I see, do you know when the production url will change? This PR alters both the UAT and production environment. |
I don't know when the old production URL will stop working, I haven't seen any notices from Windcave. The new URL is what's currently advertised: https://www.windcave.com/developer-ecommerce-pxpost |
@steveh I've reached out to Windcave to confirm the prod urls have changed. Once they respond I can approve this PR! |
@steveh Windcave has confirmed that merchants should move to the new URL! Could you please add a CHANGELOG entry? |
6bc0552
to
5567a1c
Compare
@aenand Rebased and added changelog |
@steveh ahhh someone merged an update to the CHANGELOG. If you can update it to resolve the merge conflict I will merge this first thing tomorrow morning! |
Now that Payment Express has renamed to Windcave, the old endpoints no longer work. Test Summary Local: 5543 tests, 77553 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed Unit: 37 tests, 263 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed Remote: 17 tests, 79 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
5567a1c
to
1253092
Compare
@aenand Rebased again |
Now that Payment Express has renamed to Windcave, the old endpoints no longer work.
Test Summary
Local: 5543 tests, 77553 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
Unit: 37 tests, 263 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
Remote: 17 tests, 79 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed