-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Price in cart rule#27470 #32441
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
base: 2.4-develop
Are you sure you want to change the base?
Price in cart rule#27470 #32441
Conversation
Merge from magento
pull from origin
update fork
pull origin
pull origin
pull origin
Hi @gixid192. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento run all tests |
@magento run Integration Tests, Functional Tests B2B, Functional Tests EE, WebAPI Tests |
Description (*)
Price in cart only calculates based on 1 qty
Fixed Issues (if relevant)
Questions or comments
I saw the code has 2 comments from Magento team, added long ago since 2014
/@todo reimplement this method when is fixed MAGETWO-5713
.I'm not sure if they are about to reimplement it, but due to the time and the error, I just put the code here so that it can work as expected.
possible bug: need to use $model->getBasePrice()
This is from this file created. Because it's been a long time, and none is asking about it so I think it will be alright to leave it there.
Please let me know what kind of tests should I write for this PR. Thanks!
Contribution checklist (*)