temp bug fix for group info being overwritten when loading a flow from storage

Author: devhelpr

libs/visual-programming-system/src/components/rect.ts

@@ -1567,7 +1567,10 @@ export class Rect<T extends BaseNodeInfo> {
         }
         if (this.nodeInfo?.isGroup) {
           const groupOfInitiator = initiator?.nodeInfo?.groupId;
-          if (groupOfInitiator && groupOfInitiator !== this.nodeComponent.id) {
+          if (
+            !groupOfInitiator ||
+            (groupOfInitiator && groupOfInitiator !== this.nodeComponent.id)
+          ) {
             this.nodeInfo?.groupedNodeIds?.forEach((id) => {
               const groupedNode = this.canvasApp?.elements.get(id);
               if (groupedNode && groupedNode.id !== initiator.id) {

libs/visual-programming-system/src/import/import-to-canvas.ts

@@ -245,7 +245,7 @@ export const importToCanvas = <T extends BaseNodeInfo>(
               });
             }
           } else {
-            nodeTask.createVisualNode(
+            const createdNode = nodeTask.createVisualNode(
               canvasApp,
               node.x,
               node.y,
@@ -258,6 +258,19 @@ export const importToCanvas = <T extends BaseNodeInfo>(
               node.nodeInfo,
               getNodeTaskFactory
             );
+
+            // TODO : find a better implementation for this
+            // problem now is that the nodes should pass the above node.nodeInfo themselves
+            // to the create canvasApp.createRect function call when creating the node ...
+            // ... and then it is overwritten.. the below fixes this for now
+            if (
+              createdNode &&
+              node.nodeInfo?.isInGroup &&
+              createdNode.nodeInfo
+            ) {
+              createdNode.nodeInfo.isInGroup = true;
+              createdNode.nodeInfo.groupId = node.nodeInfo.groupId;
+            }
           }
         }
       }

libs/web-flow-executor/src/nodes/group.ts

@@ -49,7 +49,7 @@ export const getGroup: NodeTaskFactory<NodeInfo> = (
       const wrapper = createElement(
         'div',
         {
-          class: `${cssClasses.wrapperGroupCssClasses} bg-purple-500`,
+          class: `${cssClasses.wrapperGroupCssClasses} bg-slate-700`,
         },
         undefined
       ) as unknown as INodeComponent<NodeInfo>;