@trackunit/react-graphql-hooks 1.3.24 → 1.3.29

package/index.cjs.js

@@ -136,7 +136,7 @@ const unionEdgesByNodeKey = (key, previousEdges, newEdges) => {
 const usePaginationQuery = (document, { ...props }) => {
     const [lastFetchedData, setLastFetchedData] = react.useState();
     const [resetTrigger, setResetTrigger] = react.useState(0);
-    const [abortController, setAbortController] = react.useState(new AbortController());
+    const [abortController] = react.useState(new AbortController());
     // Makes **sure** query variables are stable.
     // Before, it required variables to always be memorized in the parent which was easy to forget and confusing.
     // Maybe better solution exists but this is the best I could come up with without changing to much.
@@ -264,16 +264,6 @@ const usePaginationQuery = (document, { ...props }) => {
         }
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [after]);
-    react.useEffect(() => {
-        return () => {
-            abortController.abort();
-            setAbortController(new AbortController());
-        };
-    }, 
-    // Due to issues with StritMode and useEffect cleanup, we need to disable the exhaustive-deps rule here.
-    // For some reason, when the rule is enabled, the cleanup function is not called when the component is unmounted properly.
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-    []);
     return react.useMemo(() => {
         return {
             data,

package/index.esm.js

@@ -115,7 +115,7 @@ const unionEdgesByNodeKey = (key, previousEdges, newEdges) => {
 const usePaginationQuery = (document, { ...props }) => {
     const [lastFetchedData, setLastFetchedData] = useState();
     const [resetTrigger, setResetTrigger] = useState(0);
-    const [abortController, setAbortController] = useState(new AbortController());
+    const [abortController] = useState(new AbortController());
     // Makes **sure** query variables are stable.
     // Before, it required variables to always be memorized in the parent which was easy to forget and confusing.
     // Maybe better solution exists but this is the best I could come up with without changing to much.
@@ -243,16 +243,6 @@ const usePaginationQuery = (document, { ...props }) => {
         }
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [after]);
-    useEffect(() => {
-        return () => {
-            abortController.abort();
-            setAbortController(new AbortController());
-        };
-    }, 
-    // Due to issues with StritMode and useEffect cleanup, we need to disable the exhaustive-deps rule here.
-    // For some reason, when the rule is enabled, the cleanup function is not called when the component is unmounted properly.
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-    []);
     return useMemo(() => {
         return {
             data,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/react-graphql-hooks",
-  "version": "1.3.24",
+  "version": "1.3.29",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {
@@ -8,11 +8,11 @@
   },
   "dependencies": {
     "@apollo/client": "3.10.4",
-    "react": "18.3.1",
+    "react": "19.0.0",
     "lodash": "4.17.21",
-    "@trackunit/i18n-library-translation": "1.3.24",
-    "@trackunit/shared-utils": "1.5.23",
-    "@trackunit/react-table-pagination": "1.3.23"
+    "@trackunit/i18n-library-translation": "1.3.29",
+    "@trackunit/shared-utils": "1.5.28",
+    "@trackunit/react-table-pagination": "1.3.28"
   },
   "module": "./index.esm.js",
   "main": "./index.cjs.js",