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

Tax calculation disregards discounts [CRITICAL] #129

Closed
pukhalski opened this issue Sep 11, 2018 · 4 comments · Fixed by #155
Closed

Tax calculation disregards discounts [CRITICAL] #129

pukhalski opened this issue Sep 11, 2018 · 4 comments · Fixed by #155
Assignees

Comments

@pukhalski
Copy link

- Do you want to request a feature or report a bug?
Bug.

- What is the current behavior?
The total tax stays the same even if a discount code is applied
(while it actually should be based to the final price, including the
discount)

- What is the expected behavior?
Tax calculation should be based to the final price, including the
discount

Duplicate of netlify/gocommerce-js#16

@indysigner
Copy link

indysigner commented Oct 5, 2018

Hi @phae, do we have any news on this? The total tax still stays the same even if a discount code is applied. See the below screenshots for comparison! + @biilmann @ChrBach @pukhalski

Checkout with discount:

shop-with-discount

Checkout without discount:
shop-without-discount

@mraerino
Copy link
Contributor

Quick heads up on this one, since it seems rather important:
The price calculation logic needs to be refined in some places, so this is not easy to fix.
I will definitely get it done in the beginning of next week, so expect this to be fixed next week.

@indysigner
Copy link

I was just setting up two typical examples of how we may want to reroute tax calculation with fixed and relative discounts: (link) @mraerino

@bcomnes
Copy link
Contributor

bcomnes commented Nov 15, 2018

I didn't mean to reopen this. Not sure how that happened.

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

Successfully merging a pull request may close this issue.

5 participants