-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
fix: add reservations for current line item to available quantity #10801
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: v1.x
Are you sure you want to change the base?
fix: add reservations for current line item to available quantity #10801
Conversation
|
@kevinfurmanski is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
Does it require to update the tests? |
I don' think so, it's a UI issue in admin |
f357bc2
to
d379aea
Compare
@olivermrbl Can we get this merged to the v1.x branch and make a new release? I currently need to run a custom script to mark orders as fulfilled since it's not possible to do through the admin UI 😅 |
@riqwan Thank you for approving my PR. Do you know how to get approval from medusajs/ui to be able to merge this commit? |
I would be required to create a PR for that version. |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
About to finish, we need a reviewer here. |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
@SalahAdDin @riqwan Can someone review this PR so it can finally be merged? |
@riqwan can you check it, please? |
Maybe a version for |
This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
@riqwan is there anyone planning to merge this? |
When attempting to create a fulfillment with the last remaining item in stock, the reservation of the item blocks you from creating the fulfillment. This PR adds the quantity of any reservations for the specific line item and location to the available quantity for fulfillment.
Fixes #9821