@stoker-platform/web-app 0.5.112 → 0.5.114

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @stoker-platform/web-app
 
+## 0.5.114
+
+### Patch Changes
+
+- fix: hide file name dialog on upload
+
+## 0.5.113
+
+### Patch Changes
+
+- feat: prevent file upload button re-click
+
 ## 0.5.112
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@stoker-platform/web-app",
-    "version": "0.5.112",
+    "version": "0.5.114",
     "type": "module",
     "license": "SEE LICENSE IN LICENSE.md",
     "scripts": {

package/src/Files.tsx

@@ -8,7 +8,7 @@ import {
     StorageItem,
     UploadProgress,
 } from "@stoker-platform/types"
-import { useEffect, useState, useCallback } from "react"
+import { useEffect, useState, useCallback, useRef } from "react"
 import { getCachedConfigValue, runHooks, sanitizeDownloadFilename, validateStorageName } from "@stoker-platform/utils"
 import {
     getCollectionConfigModule,
@@ -183,6 +183,7 @@ export const RecordFiles = ({ collection, record }: FilesProps) => {
     const { setIsRouteLoading } = useRouteLoading()
 
     const [showFilenameDialog, setShowFilenameDialog] = useState(false)
+    const uploadLockRef = useRef(false)
     const [pendingUploadFile, setPendingUploadFile] = useState<File | null>(null)
     const [editingFilename, setEditingFilename] = useState("")
 
@@ -519,7 +520,7 @@ export const RecordFiles = ({ collection, record }: FilesProps) => {
         [uploadFiles, shouldSkipPermissionsDialog, getDefaultPermissions],
     )
 
-    const handleConfirmFilename = useCallback(async () => {
+    const handleConfirmFilename = useCallback(() => {
         if (!pendingUploadFile) return
         const trimmedName = editingFilename.trim()
         const validationError = validateStorageName(trimmedName)
@@ -528,8 +529,14 @@ export const RecordFiles = ({ collection, record }: FilesProps) => {
             return
         }
         if (shouldSkipPermissionsDialog()) {
-            await uploadFiles([pendingUploadFile], getDefaultPermissions(), trimmedName)
+            if (uploadLockRef.current) return
+            uploadLockRef.current = true
+            setShowFilenameDialog(false)
+            uploadFiles([pendingUploadFile], getDefaultPermissions(), trimmedName).finally(() => {
+                uploadLockRef.current = false
+            })
         } else {
+            setShowFilenameDialog(false)
             setShowPermissionsDialog(true)
         }
     }, [pendingUploadFile, editingFilename, shouldSkipPermissionsDialog, uploadFiles, getDefaultPermissions])
@@ -1349,6 +1356,7 @@ export const RecordFiles = ({ collection, record }: FilesProps) => {
                                 className="mt-1"
                                 onKeyDown={(e) => {
                                     if (e.key === "Enter") {
+                                        if (!editingFilename.trim()) return
                                         handleConfirmFilename()
                                     } else if (e.key === "Escape") {
                                         handleCancelFilename()

package/src/Form.tsx

@@ -4623,8 +4623,8 @@ function RecordForm({
                                         <FormItem>
                                             <FormDescription className="pb-2">
                                                 {originalRecord?.User_ID
-                                                    ? `Enter a password below to change the password for this ${labels.record}`
-                                                    : `Add system access for this ${labels.record} by entering a password below`}
+                                                    ? `Enter a password below to change the password for this ${recordTitle}`
+                                                    : `Add system access for this ${recordTitle} by entering a password below`}
                                             </FormDescription>
                                             <FormLabel className="text-primary">
                                                 {record?.User_ID ? "New Password" : "Password"}

package/src/loadRoutes.tsx

@@ -89,6 +89,7 @@ export const loadRoutes = (): RouteObject[] => {
             element: <Dashboard />,
             errorElement: <ErrorPage />,
         })
+        // eslint-disable-next-line security/detect-object-injection
     } else if (homePage && collectionAccess("Read", permissions.collections?.[homePage])) {
         routes[0].children.unshift({
             index: true,