Hotfix: grab orders by dateOrdered instead of dateUpdated

FosterCommerce / shipstation-connect

A plugin for Craft Commerce 2 that integrates with ShipStation.

https://plugins.craftcms.com/shipstationconnect

Other

7 stars 10 forks source link

Hotfix: grab orders by dateOrdered instead of dateUpdated #27

Closed michaelramuta closed 3 years ago

sjcallender commented 3 years ago

@michaelramuta We do not want to user dateOrdered since we need to ensure orders that have been updated are resent to ShipStation.

What prompted this on your side? We're open to learning how you're using it and expecting it to work.

michaelramuta commented 3 years ago

Ah, I should've left a more descriptive comment! We migrated a site over from craft 2 to 3 and the migration process resaved/updated all the orders. Wanted to bypass the normal Shipstation process (it was tens of thousands of pages of orders updated) so I built this PR to patch the plugin temporarily. Certainly not a feature to permanently have!

sjcallender commented 3 years ago

Got it. That's helpful for us to be aware of, actually. Future Commerce updates could very well make this happen again.