medusa icon indicating copy to clipboard operation
medusa copied to clipboard

feat: Refactor the product module definitions and implementation

Open sradevski opened this issue 10 months ago • 2 comments

There are several things done in this PR, namely:

Unify the service endpoints API to always work with a model rather than allowing to pass both ID and model (eg. both type_id and type being available in the request to create). Start using upsertWithReplace to simplify the code and fix some deassociation bugs Apply some changes to tests to deal with the pricing breaking changes Correctly define the model relationships (with both ID and entity fields available) All tests for the product are passing, which should bring us back to a great baseline.

sradevski avatar Mar 28 '24 14:03 sradevski

⚠️ No Changeset found

Latest commit: 829bb7b6cccbd2cb3840e1ee4118764b8943da30

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Mar 28 '24 14:03 changeset-bot[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
medusa-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 29, 2024 8:49am
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
api-reference ⬜️ Ignored (Inspect) Mar 29, 2024 8:49am
docs-ui ⬜️ Ignored (Inspect) Visit Preview Mar 29, 2024 8:49am
medusa-docs ⬜️ Ignored (Inspect) Visit Preview Mar 29, 2024 8:49am

vercel[bot] avatar Mar 28 '24 14:03 vercel[bot]