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] Negative order total (credit owed) after changing to a shipping method without fee #11361

Closed
drummer83 opened this issue Aug 8, 2023 · 2 comments · Fixed by #11677
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

Description

Follow up of #11117.
When starting a checkout process with a voucher and with a shipping method with shipping fee and later switching to a shipping method without shipping fee the result can be a negative order total amount in the order confirmation and a credit owed in the back office.

Expected Behavior

Changes in shipping fees must be reflected correctly and must not lead to a negative order total.

Actual Behaviour

It is possible to have a negative order total.

Steps to Reproduce

  1. As enterprise user set up a shop with a voucher and two shipping methods (one with a shipping fee and one without).
  2. As customer put items in your cart with an order total less than the voucher amount.
  3. Proceed to checkout and select the shipping method with shipping fee in step 1.
  4. Continue to step 2 and finish it.
  5. In step 3 select 'Edit' next to Delivery details.
  6. Select the shipping method without shipping fee.
  7. Continue to step 2 and select a payment method (if required).
  8. Continue to step 3.
  9. Look at the order summary with the negative order total.
  10. Complete the order.
  11. As enterprise user look at the order in the back office and see that there is a 'credit owed'.

Animated Gif/Screenshot

Peek.2023-08-08.18-37.mp4

Workaround

Severity

Your Environment

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

Possible Fix

@rioug
Copy link
Collaborator

rioug commented Aug 10, 2023

I think a workaround would be to delete and re add the same voucher when coming back to step 2 after editing the shipping.

@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 29, 2023
@rioug rioug self-assigned this Oct 18, 2023
@rioug
Copy link
Collaborator

rioug commented Oct 23, 2023

This should be fixed by #11677

@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.

3 participants