Add a delay before reporting transactions

SuperGoodSoft / solidus_taxjar

Support for using TaxJar to handle tax calculations in Solidus

BSD 3-Clause "New" or "Revised" License

12 stars 12 forks source link

Add a delay before reporting transactions #255

Closed benjaminwil closed 10 months ago

benjaminwil commented 10 months ago

What is the goal of this PR?

In a test environment, we discovered that sometimes these jobs would be run before an order's new totals had been persisted by the Spree::OrderUpdater. This is an effective workaround, but it is just a workaround.

Merge Checklist

[x] Update the changelog

forkata commented 10 months ago

Looks like tests should pass here after we merge #254. Thanks for adding this @benjaminwil!