[Shipping Lines] Update ShippingMethod networking model

woocommerce / woocommerce-ios

WooCommerce iOS app

https://www.woocommerce.com/mobile

GNU General Public License v2.0

257 stars 108 forks source link

[Shipping Lines] Update ShippingMethod networking model #12637

Closed rachelmcr closed 2 weeks ago

rachelmcr commented 2 weeks ago

Why

This is a small followup to #12632 to add the siteID to the ShippingMethod networking model, so we can fetch and store shipping methods for multiple stores.

Testing instructions

This model is not yet used in the app; confirm unit tests pass.

[x] I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

dangermattic commented 2 weeks ago

	1 Warning
:warning:	This PR is assigned to the milestone 18.5. This milestone is due in less than 2 days. Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by :no_entry_sign: Danger

wpmobilebot commented 2 weeks ago

📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce iOS
	Build Number	pr12637-d60dfbb
	Version	18.4
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	d60dfbb566f479d55b3f00d55142549143466bf8
	App Center Build	WooCommerce - Prototype Builds #8935

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.