Fix(medusa, admin-ui): Order allocations

[pKorsholm]

What

• Fix delete-reservation endpoint to allow deletions of reservations
• remove inventory_item_id from reservation update call to properly update reservation
• invalidate all list caches to properly update order overview after an allocation has been created
• Fix overlap for long product titles in edit-allocation-modal

Fixes CORE-1214

[vercel]

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

1 Ignored Deployment

Name	Status	Preview	Comments	Updated
medusa-docs	⬜️ Ignored (Inspect)			Mar 9, 2023 at 4:38PM (UTC)

[changeset-bot]

🦋 Changeset detected

Latest commit: 1f413ff

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

This PR includes changesets to release 4 packages

Name	Type
@medusajs/medusa	Patch
medusa-react	Patch
@medusajs/admin-ui	Patch
@medusajs/inventory	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

[srindom]

When we update a reservation to 0, I think it would make sense to delete it instead of updating it. Unless you can think of a scenario where it makes sense to have a neutral reservation? I can't.

[srindom]

Great work - would like a second iteration on the design for long content and suggest that we delete the reservation if the reservation quantity is updated to 0. But other than those two tweaks LGTM!

[srindom]

When I delete an allocation I sometimes get two toasters in Admin:

[srindom]

LGTM - we are moving text overlap to a separate PR