fix: simplify useFramework hook by removing EventModule dependency in…

… useNavigateOnContextChange
equinor · Nov 19, 2024 · 2aaeca8 · 2aaeca8
1 parent b42a403
commit 2aaeca8
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/portal/project-portal-common/src/hooks/use-navigate-on-context-change.ts b/portal/project-portal-common/src/hooks/use-navigate-on-context-change.ts
@@ -1,7 +1,6 @@
 import { useEffect } from 'react';
 import { NavigationModule } from '@equinor/fusion-framework-module-navigation';
-import { useFramework } from '@equinor/fusion-framework-react-app/framework';
-import { EventModule } from '@equinor/fusion-framework-module-event';
+import { useFramework } from '@equinor/fusion-framework-react';
 
 import { getContextPageURL } from '../utils';
 
@@ -15,7 +14,7 @@ import { getContextPageURL } from '../utils';
  * @returns {void}
  */
 export const useNavigateOnContextChange = () => {
-  const { modules } = useFramework<[NavigationModule, EventModule]>();
+  const { modules } = useFramework<[NavigationModule]>();
 
   useEffect(() => {
     return modules.event.addEventListener('onCurrentContextChanged', (event) => {
@@ -27,5 +26,5 @@ export const useNavigateOnContextChange = () => {
 
       modules.navigation.push(url);
     });
-  }, [modules]);
+  }, []);
 };