Update useMutation memoization (#166)

owinter86 · tannerlinsley · web-flow · commit 2ac58a18c6f2 · 2020-02-27T13:38:11.000-07:00
* Update useMutation memoization

* Update useMutation.js

Co-authored-by: Tanner Linsley &lt;tannerlinsley@gmail.com&gt;
diff --git a/src/useMutation.js b/src/useMutation.js
@@ -64,12 +64,15 @@ export function useMutation(
     ...useConfigContext(),
     ...config,
   })
+  
+  const getStatus = useGetLatest(state.status)
 
   const mutate = React.useCallback(
     async (variables, options = {}) => {
-      if (![statusIdle, statusSuccess, statusError].includes(state.status)) {
+      if (![statusIdle, statusSuccess, statusError].includes(getStatus())) {
         return
       }
+      
       dispatch({ type: actionMutate })
 
       const resolvedOptions = {
@@ -95,7 +98,7 @@ export function useMutation(
         }
       }
     },
-    [getConfig, getMutationFn, state.status]
+    [getConfig, getMutationFn, getStatus]
   )
 
   const reset = React.useCallback(() => dispatch({ type: actionReset }), [])

Original file line number	Diff line number	Diff line change
`@@ -64,12 +64,15 @@ export function useMutation(`
`64`	`64`	`...useConfigContext(),`
`65`	`65`	`...config,`
`66`	`66`	`})`
	`67`	`+`
	`68`	`+ const getStatus = useGetLatest(state.status)`
`67`	`69`
`68`	`70`	`const mutate = React.useCallback(`
`69`	`71`	`async (variables, options = {}) => {`
`70`		`- if (![statusIdle, statusSuccess, statusError].includes(state.status)) {`
	`72`	`+ if (![statusIdle, statusSuccess, statusError].includes(getStatus())) {`
`71`	`73`	`return`
`72`	`74`	`}`
	`75`	`+`
`73`	`76`	`dispatch({ type: actionMutate })`
`74`	`77`
`75`	`78`	`const resolvedOptions = {`
`@@ -95,7 +98,7 @@ export function useMutation(`
`95`	`98`	`}`
`96`	`99`	`}`
`97`	`100`	`},`
`98`		`- [getConfig, getMutationFn, state.status]`
	`101`	`+ [getConfig, getMutationFn, getStatus]`
`99`	`102`	`)`
`100`	`103`
`101`	`104`	`const reset = React.useCallback(() => dispatch({ type: actionReset }), [])`