Merge pull request #427 from aldabil21/remote-fetch-deps

Remove unneeded deps from remote fetch method

Author: aldabil21

eslint.config.js

@@ -2,7 +2,7 @@ import react from "eslint-plugin-react";
 import tseslint, { configs as tseslintConfigs } from "typescript-eslint";
 import globals from "globals";
 import js from "@eslint/js";
-import reactHooks from "eslint-plugin-react-hooks";
+import * as reactHooks from "eslint-plugin-react-hooks";
 import reactRefresh from "eslint-plugin-react-refresh";
 import pluginImport from "eslint-plugin-import";
 import pluginJsxA11y from "eslint-plugin-jsx-a11y";

package.json

@@ -79,7 +79,7 @@
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-react": "^7.37.4",
-    "eslint-plugin-react-hooks": "^5.1.0",
+    "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-react-refresh": "^0.4.19",
     "globals": "^16.0.0",
     "husky": "^9.1.7",

src/lib/views/Day.tsx

@@ -94,7 +94,8 @@ const Day = () => {
     } finally {
       triggerLoading(false);
     }
-  }, [triggerLoading, START_TIME, END_TIME, getRemoteEvents, handleState]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [getRemoteEvents]);
 
   useEffect(() => {
     if (getRemoteEvents instanceof Function) {

src/lib/views/Month.tsx

@@ -63,7 +63,8 @@ const Month = () => {
     } finally {
       triggerLoading(false);
     }
-  }, [triggerLoading, eachWeekStart, daysList.length, getRemoteEvents, handleState]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [daysList.length, getRemoteEvents]);
 
   useEffect(() => {
     if (getRemoteEvents instanceof Function) {

src/lib/views/Week.tsx

@@ -69,7 +69,8 @@ const Week = () => {
     } finally {
       triggerLoading(false);
     }
-  }, [triggerLoading, getRemoteEvents, weekStart, weekEnd, handleState]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [getRemoteEvents]);
 
   useEffect(() => {
     if (getRemoteEvents instanceof Function) {