Merge pull request #39 from guendev/fix/useMutation-hook

fix: ensure onDone is triggered properly on mutation success

Author: guendev

.changeset/wide-jokes-clean.md

@@ -0,0 +1,6 @@
+---
+"@vue3-apollo/core": patch
+"@vue3-apollo/nuxt": patch
+---
+
+fix: ensure `onDone` is triggered properly on mutation success

packages/core/src/composables/useMutation.ts

@@ -7,7 +7,6 @@ import { nextTick, ref, shallowRef } from 'vue'
 import type { HookContext, UseBaseOption } from '../utils/type'
 
 import { useApolloTracking } from '../helpers/useApolloTracking'
-import { isDefined } from '../utils/isDefined'
 import { useApolloClient } from './useApolloClient'
 
 /**
@@ -115,17 +114,15 @@ export function useMutation<
                 ...mutateOptions
             })
 
-            data.value = result.data
-
-            if (isDefined(result.data)) {
-                await nextTick()
-                void onDone.trigger(result.data, { client })
-            }
-
             if (result.error) {
                 error.value = result.error
                 void onError.trigger(result.error, { client })
             }
+            else {
+                data.value = result.data
+                await nextTick()
+                void onDone.trigger(result.data as TData, { client })
+            }
 
             return result
         }