fix(dashboard): reservation inventory details #9535
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
packages/admin/dashboard/src/routes/reservations/reservation-detail/reservation-detail.tsx
Outdated
Show resolved
Hide resolved
| @@ -23,6 +24,9 @@ export const ReservationDetail = () => { | |||
| } | |||
| ) | |||
|
|
|||
| // TEMP: fetch directly since the fields are not populated with reservation call | |||
There was a problem hiding this comment.
q: where would it be more correct to fetch the item details?
There was a problem hiding this comment.
what endpoint would you suggest, this is fetching inventory item
There was a problem hiding this comment.
I am asking wrt. the comment you put here - what would be the optimal approach to make this not TEMP? Is it just adding the inventory_item field when fetching the reservation?
If so we should maybe either include that in this PR or create a separate task so that can be fixed
There was a problem hiding this comment.
Is it just adding the inventory_item field when fetching the reservation?
Yup, ideally we would want that but with this approach computed properties such as reserved_quantity are not being returned. I'll open a ticket.
What
FIXES CC-595