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

Shipping rate doesn't include tax when tax is configured to be included in prices #856

Closed
duncanmcclean opened this issue Mar 23, 2023 · 4 comments · Fixed by #874
Closed

Comments

@duncanmcclean
Copy link
Owner

duncanmcclean commented Mar 23, 2023

Description

When you have tax configured to be included in prices, it works fine for products but tax isn't included in the shipping rate applied to the cart.

A workaround for this issue would be to ensure that the price returned by your shipping method includes tax.

'tax_engine' => TaxEngine::class,
 
'tax_engine_config' => [
    // Basic Engine
    'rate'               => 10,
    'included_in_prices' => true,

    // Standard Tax Engine
    'address' => 'billing',

    'behaviour' => [
        'no_address_provided' => 'default_address',
        'no_rate_available' => 'prevent_checkout',
    ],

    'default_address' => [
        'address_line_1' => '',
        'address_line_2' => '',
        'city' => '',
        'region' => '',
        'country' => '',
        'zip_code' => '',
    ],
],

Steps to reproduce

  1. Setup a shipping method with a fixed tax rate
  2. Configure tax so it's included in prices
  3. See that the shipping price applied to the cart doesn't have tax added onto it

Environment

Unsure - reported via email.

DTD-53

@duncanmcclean duncanmcclean changed the title Shipping rate doesn't include tax when tax is configured to be included in prices [DTD-53] Shipping rate doesn't include tax when tax is configured to be included in prices Mar 23, 2023
@duncanmcclean duncanmcclean changed the title [DTD-53] Shipping rate doesn't include tax when tax is configured to be included in prices Shipping rate doesn't include tax when tax is configured to be included in prices Mar 23, 2023
@duncanmcclean
Copy link
Owner Author

This is a little more complicated than initially thought, especially for the 'standard' tax engine, as we'd need some way to figure out which of the tax rates you have setup relate to shipping.

With products, you can assign tax categories to product entries, then the rates are decided based on that & the customer's location. I need to figure out the best way to configure the category to use (maybe Simple Commerce will create a tax category just for shipping?).

@mscruse
Copy link

mscruse commented Mar 25, 2023 via email

@duncanmcclean
Copy link
Owner Author

This issue is still at the back of my mind - I've been busy the last couple of weeks with client work & getting Simple Commerce 5 ready to go.

I'll loop back to this once v5 has been released.

@duncanmcclean duncanmcclean added this to the v6.0 milestone May 16, 2023
@duncanmcclean duncanmcclean mentioned this issue May 27, 2023
2 tasks
@github-actions
Copy link

Released as part of v5.1.0.

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.

2 participants