@graphcommerce/react-hook-form 3.3.3 → 3.3.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Change Log
 
+## 3.3.5
+
+### Patch Changes
+
+- [#1665](https://github.com/graphcommerce-org/graphcommerce/pull/1665) [`1f2e14ba8`](https://github.com/graphcommerce-org/graphcommerce/commit/1f2e14ba8b674b87257a123e8cb215157890eb22) Thanks [@paales](https://github.com/paales)! - Make sure the onComplete callback also sends the variables
+
+## 3.3.4
+
+### Patch Changes
+
+- [#1660](https://github.com/graphcommerce-org/graphcommerce/pull/1660) [`75ae24a93`](https://github.com/graphcommerce-org/graphcommerce/commit/75ae24a93bd74e3b9b7efda21ec7ba6fbe9a3a75) Thanks [@paales](https://github.com/paales)! - Remove heuristicEncode for GraphQL input objects provided by react-hook-form
+
 ## 3.3.3
 
 ### Patch Changes

package/package.json

@@ -2,7 +2,7 @@
   "name": "@graphcommerce/react-hook-form",
   "homepage": "https://www.graphcommerce.org/",
   "repository": "github:graphcommerce-org/graphcommerce",
-  "version": "3.3.3",
+  "version": "3.3.5",
   "sideEffects": false,
   "prettier": "@graphcommerce/prettier-config-pwa",
   "eslintConfig": {

package/src/useFormGql.tsx

@@ -12,10 +12,7 @@ import { UseFormProps, UseFormReturn, UnpackNestedValue, DeepPartial } from 'rea
 import diff from './diff'
 import { useGqlDocumentHandler, UseGqlDocumentHandler } from './useGqlDocumentHandler'
 
-export type OnCompleteFn<Q> = (
-  data: FetchResult<Q>,
-  client: ApolloClient<unknown>,
-) => void | Promise<void>
+export type OnCompleteFn<Q, V> = (data: FetchResult<Q>, variables: V) => void | Promise<void>
 
 type UseFormGraphQLCallbacks<Q, V> = {
   /**
@@ -23,7 +20,7 @@ type UseFormGraphQLCallbacks<Q, V> = {
    * Mutation. Also allows you to send false to skip submission.
    */
   onBeforeSubmit?: (variables: V) => V | false | Promise<V | false>
-  onComplete?: OnCompleteFn<Q>
+  onComplete?: OnCompleteFn<Q, V>
 }
 
 export type UseFormGraphQlOptions<Q, V> = UseFormProps<V> & UseFormGraphQLCallbacks<Q, V>
@@ -81,7 +78,7 @@ export function useFormGql<Q, V>(
       // if (variables === false) onInvalid?.(formValues, event)
 
       const result = await execute({ variables })
-      if (onComplete && result.data) await onComplete(result, client)
+      if (onComplete && result.data) await onComplete(result, variables)
 
       // Reset the state of the form if it is unmodified afterwards
       if (typeof diff(form.getValues(), formValues) === 'undefined')

package/src/useFormGqlMutation.tsx

@@ -26,6 +26,7 @@ export function assertFormGqlOperation<V, Q = Record<string, unknown>>(
   }
 }
 
+/** Bindings between react-hook-form's useForm and Apollo Client's useMutation hook. */
 export function useFormGqlMutation<Q, V>(
   document: TypedDocumentNode<Q, V>,
   options: UseFormGraphQlOptions<Q, V> = {},

package/src/useGqlDocumentHandler.tsx

@@ -116,20 +116,8 @@ export function handlerFactory<Q, V>(document: TypedDocumentNode<Q, V>): UseGqlD
   const required = requiredPartial as Required
   const encoding = encodingPartial as Encoding
 
-  function heuristicEncode(val: string) {
-    if (Number(val).toString() === val) return Number(val)
-    if (val === 'true') return true
-    if (val === 'false') return false
-    return val
-  }
-
   function encodeItem(enc: FieldTypes, val: unknown) {
     if (Array.isArray(val)) return val.map((v, i) => encodeItem(enc[i], v))
-    if (val && typeof val === 'object') {
-      return Object.fromEntries(
-        Object.entries(val).map(([key, v]) => [key, heuristicEncode(v as string)]),
-      )
-    }
     if (enc === 'Boolean') return Boolean(val)
     if (enc === 'Float' || enc === 'Int') return Number(val)
     return val