fix: move worker setup under useEffect

grafana · Jan 7, 2025 · 1bb4a91 · 1bb4a91
1 parent 33dad64
commit 1bb4a91
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/components/App.tsx b/src/components/App.tsx
@@ -20,6 +20,10 @@ export const App = (props: AppRootProps<ProvisioningJsonData>) => {
   const { meta } = props;
 
   useEffect(() => {
+    if (process.env.NODE_ENV === 'development' && process.env.REACT_APP_MSW) {
+      setupWorker(...handlers).start();
+    }
+
     return () => {
       // we have a dependency on alerts to display our alerting correctly
       // so we are invalidating the alerts list on the assumption the user might change their alerting options when they leave SM
@@ -29,10 +33,6 @@ export const App = (props: AppRootProps<ProvisioningJsonData>) => {
     };
   }, []);
 
-  if (process.env.NODE_ENV === 'development' && process.env.REACT_APP_MSW) {
-    setupWorker(...handlers).start();
-  }
-
   return (
     <QueryClientProvider client={queryClient}>
       <MetaContextProvider meta={meta}>