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 tax rate in order summary #2150

Closed
Noodleyman opened this issue Aug 21, 2018 · 2 comments
Closed

Shipping tax rate in order summary #2150

Noodleyman opened this issue Aug 21, 2018 · 2 comments
Assignees
Milestone

Comments

@Noodleyman
Copy link

Noodleyman commented Aug 21, 2018

It would be VERY helpful to log the tax rate, and price with/without tax applied to shipping and stored against an order summary. this is helpful in case a tax rate changes over time.

Currently the only way to get the rate is to make a call to the module, and gather the tax rate, which had a dependency on the rates always remaining the same.

Having it to reference from the point of time the order was placed is valuable information.

@abrookbanks abrookbanks self-assigned this Feb 7, 2019
@abrookbanks abrookbanks added this to the 6.2.3 milestone Feb 7, 2019
@abrookbanks
Copy link
Member

Awesome for debugging!! Thanks Alec... fab suggestion.

@Dirty-Butter
Copy link

Dirty-Butter commented Feb 7, 2019

AS USUAL - the problem I asked about below is my own error. It finally dawned on me I could use Notepad++ to search the db/sql file for all instances of tax. The table field was added in 6.1.14. I'm not sure I even knew how to use the setup upgrade sql manually way back then.

ALTER TABLE CubeCart_order_inventory ADD tax_percent DECIMAL(7,4 ) NOT NULL DEFAULT '0.0000' AFTER tax; #EOQ

I don't have tax in my order_inventory structure. Did I just miss a setup change somewhere along the way? Or does altering order_inventory to add the tax field need to be included in this SQL file? Since tax is not already there in mine, the sql query fails.

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

No branches or pull requests

3 participants