Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When adding a line item promotion to an order taxes are not re-adjusted with TaxCloud. #34

Open
Lyonsclay opened this issue Jun 15, 2016 · 4 comments

Comments

@Lyonsclay
Copy link

When using the Spree::Calculator::DefaultTax an additional tax total is recalculated after applying a line item promotion to an order, however when using the Spree::Calculator::TaxCloudCalculator the taxes are not adjusted after applying the promotion.

Gemfile.lock -- https://gist.github.com/Lyonsclay/2480bf8229afc1a07a0a123343875c28

I have created a repo with a default Spree 3.0.8 app and added a test that demonstrates the behavior.

https://github.com/Lyonsclay/spree_3-0-stable_app

Clone the repo, run bundle install, and spec.

@Lyonsclay
Copy link
Author

Lyonsclay commented Jun 15, 2016

Just noticed this in the source code for TaxCloudCalculator;

TODO take discounted_amount into account. This is a problem because TaxCloud API does not take discounts nor does it return percentage rates.

@lmirosevic
Copy link

+1

@cyu
Copy link

cyu commented Jan 11, 2017

I've submitted a PR (#37) that addresses this. For 3-0-stable, but the change should work on the newer branches as well.

@cyu
Copy link

cyu commented Jan 12, 2017

#38 addresses master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants