magento-lts icon indicating copy to clipboard operation
magento-lts copied to clipboard

Add Fedex REST API rate and tracking functionality, to co-exist with Fedex WSDL

Open xqiu opened this issue 1 year ago • 2 comments

Description (*)

Add Fedex REST API rate and tracking functionality, to co-exist with Fedex WSDL. The reason is that Fedex WSDL still functions. But Fedex starts to require some accounts to use REST API (in my actual case), and stopped one of my accounts to use WSDL.

Fixed Issues (if relevant)

  1. Fixes OpenMage/magento-lts#3308

Manual testing scenarios (*)

  1. Use Fedex shipping method, get quotation from Fedex test server
  2. Use Fedex shipping method, get quotation from Fedex production server (I've used the code in production)
  3. Use Fedex shipping method, to track the shipment in openmage backend and frontend, when clicking the fedex shipment number 123456789012 for both Fedex test and productions server (I've used the code in production)

Questions or comments

The change requires PHP 8.1 and above, as the package shipstream/fedex-rest-sdk requires.

xqiu avatar Nov 21 '24 22:11 xqiu

Someone using FEDEX to test it?

sreichel avatar Jan 06 '25 22:01 sreichel

Quality Gate Failed Quality Gate failed

Failed conditions
3.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

sonarqubecloud[bot] avatar Jan 26 '25 18:01 sonarqubecloud[bot]

Figured I would try the Copilot reviewer here. FedEx will be retiring Web Services on 6/1/26 (per a recent reminder email), which I suppose was pushed back from the original May '24 date, and as @xqiu mentioned, new accounts can potentially no longer access the SOAP API. I'll keep an eye on this one and do some testing myself in the coming weeks, but if any other maintainer has an account (and the time 😄) to test it would be great to push this through.

mattdavenport avatar Aug 14 '25 19:08 mattdavenport