search

woocommerce / woocommerce-ios

WooCommerce iOS app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
302 stars 109 forks source link

[Order Details] Shipment Tracking is using an old version of the Woo REST API #7150

Open iamgabrielma opened 2 years ago

iamgabrielma commented 2 years ago

While working on a related issue I started to see these warnings in the console:

2022-06-26 22:33:00.619159+0700 WooCommerce[56118:12058270] ⚠️ You are using an older version of the Woo REST API: wc/v2, for path: orders/6732/shipment-trackings/

While it works with WooCommerce API v1, v2, and v3. In the documentation, only v3 is shown. We'll want to use the /wp-json/wc-shipment-tracking/v3/ endpoint.

sentry-io[bot] commented 2 years ago

Sentry issue: WOOCOMMERCE-IOS-1DGY

iamgabrielma commented 2 years ago

Sentry issue: WOOCOMMERCE-IOS-1DGY

Not sure if related, but cross-linking it here for reference. This seems to point to a crash after didSelectRowAt and being unable to reach the shipping endpoint:

Called from: 
UIKitCore
+0x10ba514
-[UITableView _selectRowAtIndexPath:animated:scrollPosition:notifyDelegate:isCellMultiSelect:deselectPrevious:]
2022/08/03 17:44:37:764  ⛔️ Error synchronizing tracking: No se ha podido completar la operación. (Error de Networking.DotcomError 5.)
2022/08/03 17:44:36:457  ⚠️ You are using an older version of the Woo REST API: wc/v2, for path: orders/1526/shipment-trackings/
jaclync commented 1 year ago

From the changelog of the Shipment Tracking plugin:

2020-01-14 - version 1.6.18

  • Fix - get_option calls were initiating on all page loads hindering performance.
  • Tweak - Move api to its own namespace at wc-shipment-tracking/v3/ old namespaces wc/v1 and wc/v2 are deprecated.
  • Tweak - WC 3.9 compatibility.

wc-shipment-tracking/v3 became available in v1.6.18. Two test sites I have from 2019 still have an older plugin version, it might be worth checking the % of stores with the v1.6.18+ version before updating the API endpoint to v3.