From a5d43050067290795990356ac4e1814a7c4f27d3 Mon Sep 17 00:00:00 2001 From: Christian Vidal Wolf Date: Sun, 12 Apr 2026 18:38:47 +0200 Subject: [PATCH] Fix columnUniqueValues dependency to use columns before it's defined --- package-lock.json | 7 ------- package.json | 1 - src/components/MissingDataView.tsx | 20 ++++++++++---------- 3 files changed, 10 insertions(+), 18 deletions(-) diff --git a/package-lock.json b/package-lock.json index e55404b..96511dd 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,6 @@ "@supabase/supabase-js": "^2.103.0", "@tailwindcss/vite": "^4.1.14", "@vitejs/plugin-react": "^5.0.4", - "buzz": "^2.0.0", "clsx": "^2.1.1", "dotenv": "^17.2.3", "dropbox": "^10.34.0", @@ -1931,12 +1930,6 @@ "integrity": "sha512-zRpUiDwd/xk6ADqPMATG8vc9VPrkck7T07OIx0gnjmJAnHnTVXNQG3vfvWNuiZIkwu9KrKdA1iJKfsfTVxE6NA==", "license": "BSD-3-Clause" }, - "node_modules/buzz": { - "version": "2.0.0", - "resolved": "https://registry.npmjs.org/buzz/-/buzz-2.0.0.tgz", - "integrity": "sha512-eYeTETPJp7hWUX7j3o8iJNR8VLaaWRuOYYPWTSQqA5pIxZ2g3ZJUdyjfNZnGvr85IDhfr4ONQQflGp8+MC034A==", - "license": "MIT" - }, "node_modules/bytes": { "version": "3.1.2", "resolved": "https://registry.npmjs.org/bytes/-/bytes-3.1.2.tgz", diff --git a/package.json b/package.json index 832906e..873e65a 100644 --- a/package.json +++ b/package.json @@ -15,7 +15,6 @@ "@supabase/supabase-js": "^2.103.0", "@tailwindcss/vite": "^4.1.14", "@vitejs/plugin-react": "^5.0.4", - "buzz": "^2.0.0", "clsx": "^2.1.1", "dotenv": "^17.2.3", "dropbox": "^10.34.0", diff --git a/src/components/MissingDataView.tsx b/src/components/MissingDataView.tsx index a555304..7eb5127 100644 --- a/src/components/MissingDataView.tsx +++ b/src/components/MissingDataView.tsx @@ -65,6 +65,14 @@ export function MissingDataView({ data, headers, onSaveRow, onCaptureState }: Mi const launchHeader = launchDateCol >= 0 ? headers[launchDateCol] : 'Launch Date'; const readyHeader = readyToOrderCol >= 0 ? headers[readyToOrderCol] : 'Ready to Order'; + const columns = [ + { col: COLUMNS.ARTICLE_NO, label: 'SKU', width: 100 }, + { col: COLUMNS.ARTICLE_NAME, label: 'Name', width: 220 }, + { col: COLUMNS.CLASSIFICATION, label: 'Classification', width: 130 }, + ...(launchDateCol >= 0 ? [{ col: launchDateCol, label: launchHeader, width: 130 }] : []), + ...(readyToOrderCol >= 0 ? [{ col: readyToOrderCol, label: readyHeader, width: 130 }] : []), + ]; + const columnUniqueValues = useMemo(() => { const cols = columns.map(c => c.col); const result: Record> = {}; @@ -82,14 +90,14 @@ export function MissingDataView({ data, headers, onSaveRow, onCaptureState }: Mi }); }); return result; - }, [data, launchDateCol, readyToOrderCol]); + }, [data, columns, launchDateCol, readyToOrderCol]); const getDisplayValues = (col: number) => { const values = columnUniqueValues[col]; if (!values) return []; const searchVal = filterSearch[col]?.toLowerCase() || ''; const arr = Array.from(values).sort(); - return searchVal ? arr.filter(v => v.toLowerCase().includes(searchVal)) : arr; + return searchVal ? arr.filter((v: string) => v.toLowerCase().includes(searchVal)) : arr; }; const filteredData = useMemo(() => { @@ -184,14 +192,6 @@ export function MissingDataView({ data, headers, onSaveRow, onCaptureState }: Mi { id: 'missingLaunch', label: 'Missing Launch Date' }, ]; - const columns = [ - { col: COLUMNS.ARTICLE_NO, label: 'SKU', width: 100 }, - { col: COLUMNS.ARTICLE_NAME, label: 'Name', width: 220 }, - { col: COLUMNS.CLASSIFICATION, label: 'Classification', width: 130 }, - ...(launchDateCol >= 0 ? [{ col: launchDateCol, label: launchHeader, width: 130 }] : []), - ...(readyToOrderCol >= 0 ? [{ col: readyToOrderCol, label: readyHeader, width: 130 }] : []), - ]; - const editingRow = editing ? data[editing.rowIndex] : null; return (