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

[221] Don't send cancelled or returned line items #236

Closed Noah-Silvera closed 1 year ago

Noah-Silvera commented 1 year ago

Prior to this change, each request to TaxJar sent all line items. We should exclude line items that have been returned or cancelled so the total of the line items still adds up to the order's payment total.

nvandoorn commented 1 year ago

Latest revision rebases this on-top of master and moves the repeated query into a private method.