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

Adding a shipping method to an order as an admin should be VAT excl. #5368

Closed
RachL opened this issue May 5, 2020 · 11 comments
Closed

Adding a shipping method to an order as an admin should be VAT excl. #5368

RachL opened this issue May 5, 2020 · 11 comments
Assignees
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround.

Comments

@RachL
Copy link
Contributor

RachL commented May 5, 2020

Description

When adding a fixed fee to a shipping method, this fee is added Tax included.

Yet when selecting as an admin a shipping method with fees, the amount is showing fees + VAT, which is not correct

Expected Behavior

Fees seen by the customer should be the same as those seen by the manager.

Actual Behaviour

OFN shopfront:

image.png

OFN backoffice:

image.png

Steps to Reproduce

  1. Create a shipping method with fees with a fixed amount
  2. Make an order in front office, see the amount is correct
  3. Make an order from the backoffice, see the amount is higher.

Workaround

I guess the work around is to still select the shipping method and then add an adjustment removing the incorrect amount? Not sure if this doesn't screw the VAT report...

Severity

bug-s3: a feature is broken but there is a workaround

Your Environment

  • Version used: v2.9.7
  • Browser name and version: Firefox 76
  • Operating System and version (desktop or mobile): Desktop
@RachL RachL added the bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround. label May 5, 2020
@JasmeetLuthra
Copy link

JasmeetLuthra commented Sep 2, 2020

Hi @RachL . Is the issue still there?
I tried reproducing the issue using the described steps but the shipping amount displayed is same in both front & back office.

@RachL
Copy link
Contributor Author

RachL commented Sep 3, 2020

hi @JasmeetLuthra yes the issue is still here but you need to assign a tax to your shipping method. It's something done here: https://ofn-user-guide.gitbook.io/ofn-super-admin-guide/ofn-platform-configuration/taxes-and-tax-zones#tax-settings

Let me know if this helps!

@JasmeetLuthra
Copy link

Thanks @RachL I reproduced the issue after assigning tax. Can i start working on this?

@sigmundpetersen
Copy link
Contributor

Yes you're welcome to start working on it and submit your PR on this! 👍 Thank you.
Please have a look at the readme and getting started sections if you haven't already!

@RachL
Copy link
Contributor Author

RachL commented Sep 4, 2020

yes! welcome on board @JasmeetLuthra :)

@luisramos0
Copy link
Contributor

Hello @JasmeetLuthra welcome to OFN, I recommend you look at good-first-issue label or the welcome new developers board.
I dont think this is a good first issue, actually I think this is a bad-first-issue :-) code related to taxes is tricky in OFN. Off course you can try!!! Good luck either way!

@JasmeetLuthra
Copy link

Hello @JasmeetLuthra welcome to OFN, I recommend you look at good-first-issue label or the welcome new developers board.
I dont think this is a good first issue, actually I think this is a bad-first-issue :-) code related to taxes is tricky in OFN. Off course you can try!!! Good luck either way!

Hello @luisramos0 ! Thank you for your insights . Yes, I am finding it quite hard to understand the taxation on shipping price. Whatever setting you do the price of shipment remains same . Even if I doesn't select "tax included in price" the shipment price remains same during the final checkout.
If that's how it works , then changing the shipping_rate's 'display_price' to 'cost' in shipping partial shows shipment price without any tax. But I don't know if that's how it should work so I didn't make any PR.
Yes I think I should move to good first issues first to have the better understanding of the codebase first.
Thanks

@luisramos0
Copy link
Contributor

you are welcome @JasmeetLuthra
I think the change will be during checkout workflow on adjustment.rb where there is some logic to include/exclude taxes from adjustments, in this case, shipping fees.

@filipefurtad0
Copy link
Contributor

filipefurtad0 commented Mar 11, 2021

Hey @RachL ,

I've observed this recently as well:

Peek 2021-03-11 10-47

In the example above, this shipping fee is displayed as 4,8 £ but contributes to the order amount with as 4 £ which is correct. So the extra 0,8 £ do not add up to the order total (which is a good thing).

In the example you report, do the 0,33€ add up to the order total?

@RachL RachL added the FR Selected to be done by the French active instance label Feb 25, 2022
@apricot12 apricot12 self-assigned this Mar 15, 2022
@apricot12
Copy link
Contributor

I still cant reproduce this, can you confirm this is still an issue? @RachL

@RachL
Copy link
Contributor Author

RachL commented Mar 21, 2022

@filipefurtad0 @apricot12

It was changing the order total, that's why it was super annoying. But indeed this seemed to be fixed. Anyone know what this switch does? Couldn't find it in the user guide

image

@RachL RachL closed this as completed Mar 21, 2022
@RachL RachL removed the FR Selected to be done by the French active instance label Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-s3 The bug is stopping a critical or non-critical feature but there is a usable workaround.
Projects
None yet
Development

No branches or pull requests

6 participants