Merge pull request #72 from Asciant/useEffect-array-not-object

brauliodiez · web-flow · commit b6ab0fffb922 · 2019-08-12T08:22:29.000+02:00
useEffect API accepts an array of values, not an object (config)
diff --git a/src/trackerHook.js b/src/trackerHook.js
@@ -18,7 +18,7 @@ export const usePromiseTracker = (outerConfig = defaultConfig) => {
       setInternalPromiseInProgress(true);
       setPromiseInProgress(true);
     }
-  }, config)
+  }, [config])
 
   // Internal will hold the current value
   const [

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ export const usePromiseTracker = (outerConfig = defaultConfig) => {`
`18`	`18`	`setInternalPromiseInProgress(true);`
`19`	`19`	`setPromiseInProgress(true);`
`20`	`20`	`}`
`21`		`- }, config)`
	`21`	`+ }, [config])`
`22`	`22`
`23`	`23`	`// Internal will hold the current value`
`24`	`24`	`const [`