Skip to content

Commit a98be59

Browse files
authored
chore: Reset state when re-checking for responses (#6397)
* Reset state when re-checking * Create a reset state action for processingCompleted
1 parent b4d6ffe commit a98be59

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

app/(gcforms)/[locale]/(form administration)/form-builder/[id]/responses-pilot/context/ResponsesContext.tsx

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ interface ResponsesContextType {
4343
format?: "csv" | "html"
4444
) => Promise<void>;
4545
processingCompleted: boolean;
46-
setProcessingCompleted: Dispatch<SetStateAction<boolean>>;
46+
resetProcessingCompleted: () => void;
4747
selectedFormat: string;
4848
setSelectedFormat: Dispatch<SetStateAction<string>>;
4949
interrupt: boolean;
@@ -148,10 +148,9 @@ export const ResponsesProvider = ({
148148
setNewFormSubmissions([]);
149149
};
150150

151-
// const onInterrupt = () => {
152-
// logger.info("Processing interrupted by user");
153-
// setNewFormSubmissions([]);
154-
// }
151+
const resetProcessingCompleted = () => {
152+
setProcessingCompleted(false);
153+
};
155154

156155
const processResponses = useCallback(
157156
async (initialSubmissions?: NewFormSubmission[]) => {
@@ -277,7 +276,7 @@ export const ResponsesProvider = ({
277276
setDirectoryHandle(null);
278277
setNewFormSubmissions(null);
279278
setProcessedSubmissionIds(new Set());
280-
setProcessingCompleted(false);
279+
resetProcessingCompleted();
281280
setSelectedFormat("csv");
282281
setInterrupt(false);
283282
interruptRef.current = false;
@@ -301,7 +300,7 @@ export const ResponsesProvider = ({
301300
setProcessedSubmissionIds,
302301
processResponses,
303302
processingCompleted,
304-
setProcessingCompleted,
303+
resetProcessingCompleted,
305304
selectedFormat,
306305
setSelectedFormat,
307306
interrupt: isProcessingInterrupted,

app/(gcforms)/[locale]/(form administration)/form-builder/[id]/responses-pilot/result/Confirmation.tsx

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,16 @@ export const Confirmation = ({ locale, id }: { locale: string; id: string }) =>
2020
retrieveResponses,
2121
processedSubmissionIds,
2222
setProcessedSubmissionIds,
23-
setProcessingCompleted,
23+
resetProcessingCompleted,
2424
setInterrupt,
2525
processResponses,
2626
newFormSubmissions,
2727
} = useResponsesContext();
2828

2929
const handleCheckResponses = useCallback(() => {
3030
setHasCheckedForResponses(true);
31+
resetProcessingCompleted();
32+
// eslint-disable-next-line react-hooks/exhaustive-deps
3133
}, []);
3234

3335
const handleGoBack = () => {
@@ -37,7 +39,7 @@ export const Confirmation = ({ locale, id }: { locale: string; id: string }) =>
3739
const handleSelectNewLocation = () => {
3840
// reset relevant state
3941
setProcessedSubmissionIds(new Set());
40-
setProcessingCompleted(true);
42+
resetProcessingCompleted();
4143
setInterrupt(false);
4244

4345
// navigate to location selection with reset param
@@ -47,7 +49,7 @@ export const Confirmation = ({ locale, id }: { locale: string; id: string }) =>
4749
const handleDownload = async () => {
4850
// reset relevant state
4951
setProcessedSubmissionIds(new Set());
50-
setProcessingCompleted(true);
52+
resetProcessingCompleted();
5153
setInterrupt(false);
5254

5355
const initialResponses = await retrieveResponses();

0 commit comments

Comments
 (0)