Fix: Refactor save node shortcut to avoid override modal opening when it shouldnt

src/frontend/src/pages/FlowPage/components/nodeToolbarComponent/index.tsx

@@ -211,7 +211,7 @@ export default function NodeToolbarComponent({
 
   const [openModal, setOpenModal] = useState(false);
   const hasCode = Object.keys(data.node!.template).includes("code");
-
+  
   useEffect(() => {
     function onKeyDown(event: KeyboardEvent) {
       if (
@@ -273,22 +273,17 @@ export default function NodeToolbarComponent({
       if (
         selected &&
         (event.ctrlKey || event.metaKey) &&
-        event.key === "s" &&
+        event.key === "s"
-        isSaved
       ) {
-        event.preventDefault();
+        if (isSaved) {
-        return setShowOverrideModal((state) => !state);
+          event.preventDefault();
-      }
+          return setShowOverrideModal((state) => !state);
-      if (
+        }
-        selected &&
+        if (hasCode) {
-        (event.ctrlKey || event.metaKey) &&
+          event.preventDefault();
-        event.key === "s" &&
+          saveComponent(cloneDeep(data), false);
-        hasCode
+          setSuccessData({title: `${data.id} saved successfully`})
-      ) {
+        }
-        event.preventDefault();
-        saveComponent(cloneDeep(data), false);
-        setSuccessData({title: `${data.id} saved successfully`})
-        unselectAll();
       }
       if (
         selected &&
@@ -307,7 +302,7 @@ export default function NodeToolbarComponent({
     return () => {
       document.removeEventListener("keydown", onKeyDown);
     };
-  }, []);
+  }, [isSaved]);
 
   return (
     <>