-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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 item to cart via API does not apply custom price set #38985
Comments
Hi @Lex-Weeks. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
Hi @engcom-Bravo. Thank you for working on this issue.
|
@magento give me 2.4-develop instance |
Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Bravo, here is your Magento Instance: https://1c05d0cc8d4428e93f005ab0da3dd2d5.instances-prod.magento-community.engineering |
Hi @Lex-Weeks, Thanks for your reporting and collaboration. AS per the Magento devdocs https://developer.adobe.com/commerce/webapi/rest/tutorials/orders/order-add-items/ the payload will be without price field if we are adding the price field we are getting Decoding Error. It seems to be an expected behaviour of Magento. Hence we are considering this as a improvement and marking this as feature request. Thanks. |
@engcom-Bravo You have an error in the payload. I confirm the issue occurs also in version 2.4.6. I am having the same problem, the price is not updated. |
@ignaciomnzz I have talked to adobe commercial support and they effectively said that they are not going to fix this. Took a different path in writing our own endpoint that leverages the built in item functions to do this. $item->setCustomPrice($YourPriceHere);
$item->setOriginalCustomPrice($YourPriceHere); |
Summary
When you add an item to a cart via the API, if you specify a price with the item, the cart does not honor the price you supplied.
Examples
Steps to reproduce:
Proposed solution
No response
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: