Skip to content

Commit 2003e41

Browse files
AntoLCAntoLC
committed
🚨(frontend) adapt signatures to @tanstack/react-query to >5.90
Recent upgrade of @tanstack/react-query to version >5.90 introduced a breaking change in the onSuccess and onError callback signatures for the useMutation hook. The context parameter has been replaced with an onMutateResult parameter, which provides information about the result of the onMutate callback.
1 parent 5ebdf4b commit 2003e41

File tree

9 files changed

+34
-34
lines changed

9 files changed

+34
-34
lines changed

‎src/frontend/apps/impress/src/features/docs/doc-management/api/useDuplicateDoc.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ export function useDuplicateDoc(options?: DuplicateDocOptions) {
8080

8181
return await duplicateDoc(variables);
8282
},
83-
onSuccess: (data, variables, context) => {
83+
onSuccess: (data, variables, onMutateResult, context) => {
8484
void queryClient.resetQueries({
8585
queryKey: [KEY_LIST_DOC],
8686
});
@@ -89,14 +89,14 @@ export function useDuplicateDoc(options?: DuplicateDocOptions) {
8989
duration: 3000,
9090
});
9191

92-
void options?.onSuccess?.(data, variables, context);
92+
void options?.onSuccess?.(data, variables, onMutateResult, context);
9393
},
94-
onError: (error, variables, context) => {
94+
onError: (error, variables, onMutateResult, context) => {
9595
toast(t('Failed to duplicate the document...'), VariantType.ERROR, {
9696
duration: 3000,
9797
});
9898

99-
void options?.onError?.(error, variables, context);
99+
void options?.onError?.(error, variables, onMutateResult, context);
100100
},
101101
});
102102
}

‎src/frontend/apps/impress/src/features/docs/doc-management/api/useRemoveDoc.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,19 +33,19 @@ export const useRemoveDoc = ({
3333
return useMutation<void, APIError, RemoveDocProps>({
3434
mutationFn: removeDoc,
3535
...options,
36-
onSuccess: (data, variables, context) => {
36+
onSuccess: (data, variables, onMutateResult, context) => {
3737
listInvalidQueries?.forEach((queryKey) => {
3838
void queryClient.invalidateQueries({
3939
queryKey: [queryKey],
4040
});
4141
});
4242
if (options?.onSuccess) {
43-
void options.onSuccess(data, variables, context);
43+
void options.onSuccess(data, variables, onMutateResult, context);
4444
}
4545
},
46-
onError: (error, variables, context) => {
46+
onError: (error, variables, onMutateResult, context) => {
4747
if (options?.onError) {
48-
void options.onError(error, variables, context);
48+
void options.onError(error, variables, onMutateResult, context);
4949
}
5050
},
5151
});

‎src/frontend/apps/impress/src/features/docs/doc-management/api/useRestoreDoc.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,19 +36,19 @@ export const useRestoreDoc = ({
3636
return useMutation<void, APIError, RestoreDocProps>({
3737
mutationFn: restoreDoc,
3838
...options,
39-
onSuccess: (data, variables, context) => {
39+
onSuccess: (data, variables, onMutateResult, context) => {
4040
listInvalidQueries?.forEach((queryKey) => {
4141
void queryClient.invalidateQueries({
4242
queryKey: [queryKey],
4343
});
4444
});
4545
if (options?.onSuccess) {
46-
void options.onSuccess(data, variables, context);
46+
void options.onSuccess(data, variables, onMutateResult, context);
4747
}
4848
},
49-
onError: (error, variables, context) => {
49+
onError: (error, variables, onMutateResult, context) => {
5050
if (options?.onError) {
51-
void options.onError(error, variables, context);
51+
void options.onError(error, variables, onMutateResult, context);
5252
}
5353
},
5454
});

‎src/frontend/apps/impress/src/features/docs/doc-management/api/useUpdateDoc.tsx‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,26 +42,26 @@ export function useUpdateDoc(queryConfig?: UseUpdateDoc) {
4242
return useMutation<Doc, APIError, UpdateDocParams>({
4343
mutationFn: updateDoc,
4444
...queryConfig,
45-
onSuccess: (data, variables, context) => {
45+
onSuccess: (data, variables, onMutateResult, context) => {
4646
queryConfig?.listInvalideQueries?.forEach((queryKey) => {
4747
void queryClient.invalidateQueries({
4848
queryKey: [queryKey],
4949
});
5050
});
5151

5252
if (queryConfig?.onSuccess) {
53-
void queryConfig.onSuccess(data, variables, context);
53+
void queryConfig.onSuccess(data, variables, onMutateResult, context);
5454
}
5555
},
56-
onError: (error, variables, context) => {
56+
onError: (error, variables, onMutateResult, context) => {
5757
// If error it means the user is probably not allowed to edit the doc
5858
// so we invalidate the canEdit query to update the UI accordingly
5959
void queryClient.invalidateQueries({
6060
queryKey: [KEY_CAN_EDIT],
6161
});
6262

6363
if (queryConfig?.onError) {
64-
queryConfig.onError(error, variables, context);
64+
queryConfig.onError(error, variables, onMutateResult, context);
6565
}
6666
},
6767
});

‎src/frontend/apps/impress/src/features/docs/doc-share/api/useDeleteDocAccess.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ export const useDeleteDocAccess = (options?: UseDeleteDocAccessOptions) => {
4545
return useMutation<void, APIError, DeleteDocAccessProps>({
4646
mutationFn: deleteDocAccess,
4747
...options,
48-
onSuccess: (data, variables, context) => {
48+
onSuccess: (data, variables, onMutateResult, context) => {
4949
void queryClient.invalidateQueries({
5050
queryKey: [KEY_LIST_DOC_ACCESSES],
5151
});
@@ -63,7 +63,7 @@ export const useDeleteDocAccess = (options?: UseDeleteDocAccessOptions) => {
6363
queryKey: [KEY_LIST_USER],
6464
});
6565
if (options?.onSuccess) {
66-
void options.onSuccess(data, variables, context);
66+
void options.onSuccess(data, variables, onMutateResult, context);
6767
}
6868
},
6969
});

‎src/frontend/apps/impress/src/features/docs/doc-share/api/useDeleteDocInvitation.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -53,17 +53,17 @@ export const useDeleteDocInvitation = (
5353
>({
5454
mutationFn: deleteDocInvitation,
5555
...options,
56-
onSuccess: (data, variables, context) => {
56+
onSuccess: (data, variables, onMutateResult, context) => {
5757
void queryClient.invalidateQueries({
5858
queryKey: [KEY_LIST_DOC_INVITATIONS],
5959
});
6060
if (options?.onSuccess) {
61-
void options.onSuccess(data, variables, context);
61+
void options.onSuccess(data, variables, onMutateResult, context);
6262
}
6363
},
64-
onError: (error, variables, context) => {
64+
onError: (error, variables, onMutateResult, context) => {
6565
if (options?.onError) {
66-
void options.onError(error, variables, context);
66+
void options.onError(error, variables, onMutateResult, context);
6767
}
6868
},
6969
});

‎src/frontend/apps/impress/src/features/docs/doc-share/api/useDocAccessRequest.tsx‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,12 +59,12 @@ export function useCreateDocAccessRequest(
5959
return useMutation<void, APIError, CreateDocAccessRequestParams>({
6060
mutationFn: createDocAccessRequest,
6161
...options,
62-
onSuccess: (data, variables, context) => {
62+
onSuccess: (data, variables, onMutateResult, context) => {
6363
void queryClient.resetQueries({
6464
queryKey: [KEY_LIST_DOC_ACCESS_REQUESTS],
6565
});
6666

67-
void options?.onSuccess?.(data, variables, context);
67+
void options?.onSuccess?.(data, variables, onMutateResult, context);
6868
},
6969
});
7070
}
@@ -169,7 +169,7 @@ export const useAcceptDocAccessRequest = (
169169
return useMutation<void, APIError, acceptDocAccessRequestsParams>({
170170
mutationFn: acceptDocAccessRequests,
171171
...options,
172-
onSuccess: (data, variables, context) => {
172+
onSuccess: (data, variables, onMutateResult, context) => {
173173
void queryClient.invalidateQueries({
174174
queryKey: [KEY_LIST_DOC_ACCESSES],
175175
});
@@ -179,7 +179,7 @@ export const useAcceptDocAccessRequest = (
179179
});
180180

181181
if (options?.onSuccess) {
182-
void options.onSuccess(data, variables, context);
182+
void options.onSuccess(data, variables, onMutateResult, context);
183183
}
184184
},
185185
});
@@ -223,13 +223,13 @@ export const useDeleteDocAccessRequest = (
223223
return useMutation<void, APIError, DeleteDocAccessRequestParams>({
224224
mutationFn: deleteDocAccessRequest,
225225
...options,
226-
onSuccess: (data, variables, context) => {
226+
onSuccess: (data, variables, onMutateResult, context) => {
227227
void queryClient.invalidateQueries({
228228
queryKey: [KEY_LIST_DOC_ACCESS_REQUESTS],
229229
});
230230

231231
if (options?.onSuccess) {
232-
void options.onSuccess(data, variables, context);
232+
void options.onSuccess(data, variables, onMutateResult, context);
233233
}
234234
},
235235
});

‎src/frontend/apps/impress/src/features/docs/doc-share/api/useUpdateDocAccess.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ export const useUpdateDocAccess = (options?: UseUpdateDocAccessOptions) => {
5050
return useMutation<Access, APIError, UpdateDocAccessProps>({
5151
mutationFn: updateDocAccess,
5252
...options,
53-
onSuccess: (data, variables, context) => {
53+
onSuccess: (data, variables, onMutateResult, context) => {
5454
void queryClient.invalidateQueries({
5555
queryKey: [KEY_LIST_DOC_ACCESSES],
5656
});
@@ -65,7 +65,7 @@ export const useUpdateDocAccess = (options?: UseUpdateDocAccessOptions) => {
6565
queryKey: [KEY_LIST_DOC],
6666
});
6767
if (options?.onSuccess) {
68-
void options.onSuccess(data, variables, context);
68+
void options.onSuccess(data, variables, onMutateResult, context);
6969
}
7070
},
7171
});

‎src/frontend/apps/impress/src/features/docs/doc-share/api/useUpdateDocInvitation.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,17 +62,17 @@ export const useUpdateDocInvitation = (
6262
>({
6363
mutationFn: updateDocInvitation,
6464
...options,
65-
onSuccess: (data, variables, context) => {
65+
onSuccess: (data, variables, onMutateResult, context) => {
6666
void queryClient.invalidateQueries({
6767
queryKey: [KEY_LIST_DOC_INVITATIONS],
6868
});
6969
if (options?.onSuccess) {
70-
void options.onSuccess(data, variables, context);
70+
void options.onSuccess(data, variables, onMutateResult, context);
7171
}
7272
},
73-
onError: (error, variables, context) => {
73+
onError: (error, variables, onMutateResult, context) => {
7474
if (options?.onError) {
75-
void options.onError(error, variables, context);
75+
void options.onError(error, variables, onMutateResult, context);
7676
}
7777
},
7878
});

0 commit comments

Comments
 (0)