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

[Vouchers] Excl. tax is calculated incorrectly when moving back in checkout process #11359

Closed
drummer83 opened this issue Aug 8, 2023 · 1 comment · Fixed by #11677
Closed
Assignees
Labels
bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround.

Comments

@drummer83
Copy link
Contributor

drummer83 commented Aug 8, 2023

Description

Follow up of #11117.
There are different tax calculations depending on the way the checkout process is finished. This requires a product with taxes excluded from price and a payment method with transaction fee.

EDIT: This also seems to happen for tax incl. in price.

  • Finishing all three checkout steps in a row while adding a voucher in step 2:
    • CORRECT: Tax on voucher is calculated including the transaction fee.
      image
  • Leaving step 2 without adding the voucher and then going back from step 3 to step 2 to add the voucher yields this:
    • NOT CORRECT: Tax on voucher is calculated excluding the transaction fee.
      image

Expected Behavior

Both ways should provide identical results.
Both ways should provide the result shown as 'correct' above.

Actual Behaviour

Both ways provide different results.
The second way provides the result shown as 'not correct' above..

Steps to Reproduce

  1. As enterprise user set up a shop with a voucher, a product with tax excluded from price and a payment method with transaction fee.

Way one:

  1. As customer put that product in your cart (make sure your order total is higher than the voucher amount) and proceed to checkout.
  2. Finish step 1 of the checkout.
  3. In step 2 add the voucher code and select the payment method with transaction fees.
  4. Proceed to step 3 and check the order summary.

Way two:

  1. As customer put that product in your cart (make sure your order total is higher than the voucher amount) and proceed to checkout.
  2. Finish step 1 of the checkout.
  3. In step 2 DO NOT add the voucher code but select the payment method with transaction fees.
  4. Proceed to step 3.
  5. Bo back to step 2.
  6. Add the voucher code and select the payment method with transaction fees.
  7. Proceed to step 3 and check the order summary.

Animated Gif/Screenshot

See images above.

Workaround

Severity

Your Environment

  • Version used:
  • Browser name and version:
  • Operating System and version (desktop or mobile):

Possible Fix

@github-project-automation github-project-automation bot moved this to All the things in OFN Delivery board Aug 8, 2023
@RachL RachL added the bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround. label Sep 26, 2023
@rioug rioug self-assigned this Sep 28, 2023
@drummer83
Copy link
Contributor Author

Hi @rioug,
I just realized that this also happens for tax incl. in price actually.
I added an 'EDIT' comment in the issue above.

@filipefurtad0 filipefurtad0 self-assigned this Oct 25, 2023
@filipefurtad0 filipefurtad0 added no-staging-UK A tag which does not trigger deployments, indicating a server is being used no-staging-FR A tag which does not trigger deployments, indicating a server is being used pr-staged-uk staging.openfoodnetwork.org.uk pr-staged-fr staging.coopcircuits.fr and removed no-staging-UK A tag which does not trigger deployments, indicating a server is being used no-staging-FR A tag which does not trigger deployments, indicating a server is being used pr-staged-uk staging.openfoodnetwork.org.uk pr-staged-fr staging.coopcircuits.fr labels Oct 25, 2023
@github-project-automation github-project-automation bot moved this from All the things to Done in OFN Delivery board Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s2 The bug is affecting any of the non-critical features described in S1 and there is no workaround.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants