chore: remove debug console.logs and add error handling

- Remove console.log statements from upload file dialog
- Add onError handler to display error toasts
- Update ErrorCode reference in use-file-drop

Co-authored-by: Ona <no-reply@ona.com>
This commit is contained in:
2025-11-08 18:03:15 +00:00
parent 94d6a22ab2
commit b8c46217f7
2 changed files with 5 additions and 3 deletions

View File

@@ -1,5 +1,6 @@
import type { Doc } from "@fileone/convex/dataModel" import type { Doc } from "@fileone/convex/dataModel"
import { mutationOptions } from "@tanstack/react-query" import { mutationOptions } from "@tanstack/react-query"
import { ConvexError } from "convex/values"
import { atom, useAtom, useAtomValue, useSetAtom, useStore } from "jotai" import { atom, useAtom, useAtomValue, useSetAtom, useStore } from "jotai"
import { atomEffect } from "jotai-effect" import { atomEffect } from "jotai-effect"
import { atomWithMutation } from "jotai-tanstack-query" import { atomWithMutation } from "jotai-tanstack-query"
@@ -88,7 +89,6 @@ function useUploadFilesAtom({
) )
}, },
}).catch((error) => { }).catch((error) => {
console.log("error", error)
store.set( store.set(
fileUploadStatusAtomFamily(pickedFile.id), fileUploadStatusAtomFamily(pickedFile.id),
{ {
@@ -130,6 +130,9 @@ function useUploadFilesAtom({
toast.success("All files uploaded successfully") toast.success("All files uploaded successfully")
} }
}, },
onError: (error) => {
toast.error(formatError(error))
},
}), }),
[uploadFile, store.set], [uploadFile, store.set],
) )
@@ -533,7 +536,6 @@ function PickedFileItem({
}) { }) {
const fileUploadAtom = fileUploadStatusAtomFamily(pickedFile.id) const fileUploadAtom = fileUploadStatusAtomFamily(pickedFile.id)
const fileUpload = useAtomValue(fileUploadAtom) const fileUpload = useAtomValue(fileUploadAtom)
console.log("fileUpload", fileUpload)
const { file, id } = pickedFile const { file, id } = pickedFile
let statusIndicator: React.ReactNode let statusIndicator: React.ReactNode

View File

@@ -54,7 +54,7 @@ export function useFileDrop({
errors: Err.ApplicationErrorData[] errors: Err.ApplicationErrorData[]
}) => { }) => {
const conflictCount = errors.reduce((acc, error) => { const conflictCount = errors.reduce((acc, error) => {
if (error.code === Err.Code.Conflict) { if (error.code === Err.ErrorCode.Conflict) {
return acc + 1 return acc + 1
} }
return acc return acc