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

fix(inventory): cannot migrate existing products (#6877) #6878

Merged
merged 3 commits into from
Mar 29, 2024

Conversation

gempain
Copy link
Contributor

@gempain gempain commented Mar 29, 2024

Fixes the errors encountered in #6877.

@gempain gempain requested a review from a team as a code owner March 29, 2024 08:24
Copy link

changeset-bot bot commented Mar 29, 2024

🦋 Changeset detected

Latest commit: 5d9ca3f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@medusajs/medusa Patch

Not sure what this means? Click here to learn what changesets are.

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

Copy link

vercel bot commented Mar 29, 2024

Someone is attempting to deploy a commit to the medusajs Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Member

@sradevski sradevski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add a changeset per the contribution guide and rebase to the latest changes? I'll get it merged after that

@gempain
Copy link
Contributor Author

gempain commented Mar 29, 2024

Hm, I guess those guidelines have changed since my last PRs.

@gempain
Copy link
Contributor Author

gempain commented Mar 29, 2024

@sradevski should be good now. Let me know if I need to change something. I'm not used to using changeset.

@sradevski
Copy link
Member

@gempain Should be good now, thanks! I'll get it merged as soon as the pipeline passes

@gempain
Copy link
Contributor Author

gempain commented Mar 29, 2024

@gempain Should be good now, thanks! I'll get it merged as soon as the pipeline passes

Thanks very much 🙏

@sradevski sradevski merged commit 85a27c3 into medusajs:develop Mar 29, 2024
19 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants