From b5dd009d1ac5eaa44700c656f988a9a37955492c Mon Sep 17 00:00:00 2001
From: Tom Butcher <thomasbutcher15@gmail.com>
Date: Sat, 7 Mar 2026 19:43:59 +0000
Subject: [PATCH] Refactor FileUpload component to ensure onChange is called
 consistently after file selection, improving state management for file
 uploads.

---
 src/components/Dashboard/common/FileUpload.jsx | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/components/Dashboard/common/FileUpload.jsx b/src/components/Dashboard/common/FileUpload.jsx
index 5f3687e..e4ff122 100644
--- a/src/components/Dashboard/common/FileUpload.jsx
+++ b/src/components/Dashboard/common/FileUpload.jsx
@@ -54,6 +54,7 @@ const FileUpload = ({
         currentFiles.length === 0
       : !currentFiles
     setHasNoItems(noItems)
+    onChange(currentFiles)
   }, [currentFiles, multiple])
 
   const handleFileUpload = async (file) => {
@@ -68,11 +69,9 @@ const FileUpload = ({
           // For multiple files, add to existing array
           const newFiles = [...currentFiles, uploadedFile]
           setCurrentFiles(newFiles)
-          onChange(newFiles)
         } else {
           // For single file, replace the value
           setCurrentFiles(uploadedFile)
-          onChange(uploadedFile)
         }
       }
     } catch (error) {
@@ -88,11 +87,9 @@ const FileUpload = ({
         // For multiple files, add to existing array
         const newFiles = [...currentFiles, selectedFile]
         setCurrentFiles(newFiles)
-        onChange(newFiles)
       } else {
         // For single file, replace the value
         setCurrentFiles(selectedFile)
-        onChange(selectedFile)
       }
       // Clear the selection
       setSelectedFile(null)