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

Per product taxRate #9

Open
joshuadegier opened this issue May 22, 2019 · 3 comments
Open

Per product taxRate #9

joshuadegier opened this issue May 22, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@joshuadegier
Copy link

joshuadegier commented May 22, 2019

Hey,

good work on the repo! Only just found this after forking the original repo myself and updating it to work with Laravel 5.8. I've currently also modified the package to be able to set per-product taxRates (which defaults back to config('cart.tax') in case there is non set. Only breaking change would be the Buyable interface which now requires a getBuyableTaxRate() method.

Seems a bit useless for me to have this in a separate repo if you feel like having this feature in this package as well. Feels better to bundle the powers :-)

Changes can be found here: https://github.com/PendoNL/LaravelShoppingcart

Let me know if this sounds interesting.

@bumbummen99
Copy link
Owner

Hey, sorry for the late response, real life is kind of stressfully currently. If you would like to create a pull request i will take a look on it and check if we can improve the current solution in this repository.

@joshuadegier
Copy link
Author

Hi, no problem. I get it :-)

My repo is based on the original, so I don't think I can easily do a PR right now. I remember I didn't need to do too many changes to get this working. As said, it will be a breaking change since the Buyable interface requires the model to have a getBuyableTaxRate() method.

I can spend some time on making a PR, but only if the idea seems useful of course.

@bumbummen99 bumbummen99 added the enhancement New feature or request label Aug 2, 2019
@github-actions
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the stale Inactive, will be closed soon. label Sep 26, 2020
@github-actions github-actions bot removed the stale Inactive, will be closed soon. label Mar 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants