From d335cfb4eeabd2478891cbb9fd3f7c7994a216a5 Mon Sep 17 00:00:00 2001
From: Skorpios <cosmicray23@hotmail.com>
Date: Sat, 8 Mar 2025 17:13:20 -0800
Subject: [PATCH] Got rid of unnecessary rerenders by memoizing the
 integrationstate function.

---
 .../NotificationIntegrationModal.jsx                   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/Components/NotificationIntegrationModal/NotificationIntegrationModal.jsx b/src/Components/NotificationIntegrationModal/NotificationIntegrationModal.jsx
index 16efe77d8..81f2baec0 100644
--- a/src/Components/NotificationIntegrationModal/NotificationIntegrationModal.jsx
+++ b/src/Components/NotificationIntegrationModal/NotificationIntegrationModal.jsx
@@ -1,4 +1,4 @@
-import { useState } from "react";
+import { useState, useMemo } from "react";
 import { 
   Dialog, 
   DialogContent, 
@@ -86,8 +86,8 @@ const NotificationIntegrationModal = ({
   const theme = useTheme();
   const [tabValue, setTabValue] = useState(0);
   
-  // Initialize integrations state based on available notification types
-  const initializeIntegrationsState = () => {
+  // Memoized function to initialize integrations state
+  const initialIntegrationsState = useMemo(() => {
     const state = {};
     
     notificationTypes.forEach(type => {
@@ -102,9 +102,9 @@ const NotificationIntegrationModal = ({
     });
     
     return state;
-  };
+  }, [monitor, notificationTypes]); // Only recompute when these dependencies change
   
-  const [integrations, setIntegrations] = useState(initializeIntegrationsState());
+  const [integrations, setIntegrations] = useState(initialIntegrationsState);
 
   const handleChangeTab = (event, newValue) => {
     setTabValue(newValue);