Disable exhaustive dependency warnings where required

2023-04-26 16:53:31 -04:00
parent 6fc90c1740
commit 6c6b841f41
2 changed files with 8 additions and 6 deletions
@@ -47,12 +47,13 @@ const ServerContentPage: FunctionComponent<ServerContentPageProps> = ({ view })
        };

        loadPage();
-    }, [
+    },
+    // location.state is NOT included as a dependency here since dialogs will update state while the current view stays the same
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [
        view,
        location.pathname,
        location.search
-        // location.state is NOT included as a dependency here since dialogs will update state while the current view
-        // stays the same
    ]);

    return <></>;
@@ -64,7 +64,10 @@ const ViewManagerPage: FunctionComponent<ViewManagerPageProps> = ({
        };

        loadPage();
-    }, [
+    },
+    // location.state is NOT included as a dependency here since dialogs will update state while the current view stays the same
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [
        controller,
        view,
        type,
@@ -74,8 +77,6 @@ const ViewManagerPage: FunctionComponent<ViewManagerPageProps> = ({
        transition,
        location.pathname,
        location.search
-        // location.state is NOT included as a dependency here since dialogs will update state while the current view
-        // stays the same
    ]);

    return <></>;