Skip to content

Commit 89d2dbb

Browse files
committed
fixes after CR
1 parent 920d2b9 commit 89d2dbb

3 files changed

Lines changed: 17 additions & 8 deletions

File tree

.changeset/slimy-bikes-wink.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"saleor-dashboard": patch
3+
---
4+
5+
Add missing logic for handling product variant metadata when accessed from order line modal (on order details page).

locale/defaultMessages.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -312,10 +312,6 @@
312312
"context": "channels alphabetically title",
313313
"string": "Channels from A to Z"
314314
},
315-
"/mwSjm": {
316-
"context": "modal subheader, read-only product variant metadata",
317-
"string": "This is a metadata of the variant that is being used in this ordered item"
318-
},
319315
"/n+NRO": {
320316
"string": "Note id"
321317
},
@@ -4328,10 +4324,6 @@
43284324
"PFtMy9": {
43294325
"string": "Canceled"
43304326
},
4331-
"PH4R7g": {
4332-
"context": "modal header, read-only product variant metadata",
4333-
"string": "Product variant metadata"
4334-
},
43354327
"PHT8z1": {
43364328
"context": "extension form permissions section header",
43374329
"string": "Permissions"
@@ -7219,6 +7211,10 @@
72197211
"hPB89Y": {
72207212
"string": "No invoices to be shown"
72217213
},
7214+
"hQDWIw": {
7215+
"context": "modal header, editable product variant metadata",
7216+
"string": "Product variant metadata"
7217+
},
72227218
"hS+ZjH": {
72237219
"context": "delete webhook",
72247220
"string": "Are you sure you want to delete this webhook?"
@@ -9125,6 +9121,10 @@
91259121
"context": "button",
91269122
"string": "Set end date"
91279123
},
9124+
"tquei9": {
9125+
"context": "modal subheader, editable product variant metadata",
9126+
"string": "This is a metadata of the variant that is being used in this ordered item"
9127+
},
91289128
"tsL3IW": {
91299129
"context": "gift card history message",
91309130
"string": "Gift card was sent to customer"

src/orders/components/OrderMetadataDialog/useHandleSubmit.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,17 @@ export const useHandleOrderLineMetadataSubmit = ({
3737

3838
return createMetadataUpdateHandler(
3939
initialData,
40+
// Placeholder to keep backward compatibility - we now use react-hook-form for form state management
4041
() => Promise.resolve([]),
4142
variables => updateMetadata({ variables }),
4243
variables => updatePrivateMetadata({ variables }),
4344
);
4445
}, [initialData, updateMetadata, updatePrivateMetadata]);
4546

4647
const variantSubmitHandler = useMemo(() => {
48+
// Fallback handler: If there is no variant data to submit,
49+
// return a promise resolving to an empty error array.
50+
// This is appropriate because no submission occurs, so no errors are possible.
4751
if (!initialData?.variant) {
4852
return () => Promise.resolve([]);
4953
}

0 commit comments

Comments
 (0)