fix: add reservations for current line item to available quantity

[kevinfurmanski]

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

[changeset-bot]

⚠️ No Changeset found

Latest commit: e7a655e

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

[vercel]

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

A member of the Team first needs to authorize it.

[SalahAdDin]

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

Does it require to update the tests?

[kevinfurmanski]

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

Does it require to update the tests?

I don' think so, it's a UI issue in admin

[kevinfurmanski]

@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 😅

[kevinfurmanski]

@riqwan Thank you for approving my PR. Do you know how to get approval from medusajs/ui to be able to merge this commit?

[SalahAdDin]

@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 😅

I would be required to create a PR for that version.

[github-actions]

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.

[github-actions]

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.

[github-actions]

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]

About to finish, we need a reviewer here.

[github-actions]

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.

[kevinfurmanski]

@SalahAdDin @riqwan Can someone review this PR so it can finally be merged?

[SalahAdDin]

@riqwan can you check it, please?

[SalahAdDin]

Maybe a version for 2.x works as well.

[github-actions]

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 is there anyone planning to merge this?

[github-actions]

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 is there anyone planning to merge this?

@riqwan

[willbouch]

Hi! Sorry for the late response. I am cleaning up issues and PRs slowly but surely. I will close this PR since we actually stopped merging stuff to V1. We only will merge critical vulnerabilities to V1