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

Issue in Medusa V2 Learning Resources - Unit Price cannot be added during Cart Creation but document says so #8776

Closed
2 tasks done
sidd-kulk opened this issue Aug 27, 2024 · 3 comments · Fixed by #8820
Closed
2 tasks done

Comments

@sidd-kulk
Copy link
Contributor

What Medusa version and documentation are you using?

v2

Preliminary Checks

Issue Summary

In the V2 documentation mentioned below:
https://docs.medusajs.com/v2/resources/commerce-modules/cart
https://docs.medusajs.com/v2/api/store#carts_postcarts

It shows that Cart accepts Unit Price for an item, but as per discussion with team, it does not. The documentation needs to be updated.

How can this issue be resolved?

  1. Access the documentation as mentioned in the summary above.
  2. Access the cart payload
  3. It shows Unit Price under items node.

Are you interested in working on this issue?

  • I would like to fix this issue
@shahednasser
Copy link
Member

Thank you for opening this issue @sidd-kulk ! I've opened the PR #8820 to fix the request type of the API route, which will fix the API reference once the OpenAPI Specs are generated.

Regarding the example in https://docs.medusajs.com/v2/resources/commerce-modules/cart, I've checked with the team, and providing the unit_price is accepted for the module's main service method. so, no changes will be made there.

@sidd-kulk
Copy link
Contributor Author

Thanks for your response. Does this mean that we can pass unit_price to override the price in price domain in Medusa 2.0?

@shahednasser
Copy link
Member

Not in the API route provided by Medusa.

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