Skip to content

Commit 9155956

Browse files
committed
fix(datasets-page): stop request spam when upload finishes
1 parent 7441bb3 commit 9155956

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

features/DatasetUpload/BulkUploadDropzone.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { type Dispatch, type SetStateAction, useCallback } from "react";
1+
import { type Dispatch, type SetStateAction } from "react";
22

33
import { Grid } from "@mui/material";
44

@@ -55,10 +55,10 @@ const SingleFile = ({ index, file, setFiles, files }: SingleFileProps) => {
5555
setFiles((curr) => curr.filter((fw) => fw.file !== file));
5656
};
5757

58-
const changeToDone = useCallback(() => {
58+
const changeToDone = () => {
5959
file.done = true;
6060
setFiles(mutateAtPosition(files, index, files[index]));
61-
}, [file, files, index, setFiles]);
61+
};
6262

6363
const changeMimeType = (newType: string): void => {
6464
file.mimeType = newType;

features/DatasetUpload/SingleFileUploader.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,15 +50,17 @@ export const SingleFileUploadWithProgress = ({
5050
refetchInterval: interval,
5151
},
5252
});
53+
const taskDone = task?.done;
5354

5455
useEffect(() => {
55-
if (task?.done) {
56+
if (taskDone) {
5657
setInterval(false);
5758
void queryClient
5859
.invalidateQueries({ queryKey: getGetDatasetsQueryKey() })
5960
.then(() => changeToDone());
6061
}
61-
}, [changeToDone, queryClient, task]);
62+
// eslint-disable-next-line react-hooks/exhaustive-deps
63+
}, [queryClient, taskDone]);
6264

6365
const { extensions } = useFileExtensions();
6466
const mimeLookup = useMimeTypeLookup();

0 commit comments

Comments
 (0)