Skip to content

Commit b4c633b

Browse files
committed
removed unused custom event
1 parent 19fb61b commit b4c633b

File tree

2 files changed

+3
-16
lines changed

2 files changed

+3
-16
lines changed

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2725,9 +2725,6 @@ const WorkflowContent = React.memo(() => {
27252725

27262726
const edgesToAdd: Edge[] = autoConnectEdge ? [autoConnectEdge] : []
27272727

2728-
// Skip recording these edges separately since they're part of the parent update
2729-
window.dispatchEvent(new CustomEvent('skip-edge-recording', { detail: { skip: true } }))
2730-
27312728
// Moving to a new parent container - pass both removed and added edges for undo/redo
27322729
const affectedEdges = [...edgesToRemove, ...edgesToAdd]
27332730
updateNodeParent(node.id, potentialParentId, affectedEdges)
@@ -2746,12 +2743,10 @@ const WorkflowContent = React.memo(() => {
27462743
})
27472744
)
27482745

2749-
// Now add the edges after parent update
2746+
// Add edges after parent update (skip undo recording - it's part of parent update)
27502747
if (edgesToAdd.length > 0) {
2751-
collaborativeBatchAddEdges(edgesToAdd)
2748+
collaborativeBatchAddEdges(edgesToAdd, { skipUndoRedo: true })
27522749
}
2753-
2754-
window.dispatchEvent(new CustomEvent('skip-edge-recording', { detail: { skip: false } }))
27552750
} else if (!potentialParentId && dragStartParentId) {
27562751
// Moving OUT of a subflow to canvas
27572752
// Get absolute position BEFORE removing from parent

apps/sim/hooks/use-collaborative-workflow.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ const logger = createLogger('CollaborativeWorkflow')
3232
export function useCollaborativeWorkflow() {
3333
const undoRedo = useUndoRedo()
3434
const isUndoRedoInProgress = useRef(false)
35-
const skipEdgeRecording = useRef(false)
3635
const lastDiffOperationId = useRef<string | null>(null)
3736

3837
useEffect(() => {
@@ -58,11 +57,6 @@ export function useCollaborativeWorkflow() {
5857
)
5958
}
6059

61-
const skipEdgeHandler = (e: any) => {
62-
const { skip } = e.detail || {}
63-
skipEdgeRecording.current = skip
64-
}
65-
6660
const diffOperationHandler = (e: any) => {
6761
const {
6862
type,
@@ -108,12 +102,10 @@ export function useCollaborativeWorkflow() {
108102

109103
window.addEventListener('workflow-record-move', moveHandler)
110104
window.addEventListener('workflow-record-parent-update', parentUpdateHandler)
111-
window.addEventListener('skip-edge-recording', skipEdgeHandler)
112105
window.addEventListener('record-diff-operation', diffOperationHandler)
113106
return () => {
114107
window.removeEventListener('workflow-record-move', moveHandler)
115108
window.removeEventListener('workflow-record-parent-update', parentUpdateHandler)
116-
window.removeEventListener('skip-edge-recording', skipEdgeHandler)
117109
window.removeEventListener('record-diff-operation', diffOperationHandler)
118110
}
119111
}, [undoRedo])
@@ -1000,7 +992,7 @@ export function useCollaborativeWorkflow() {
1000992

1001993
workflowStore.batchAddEdges(edges)
1002994

1003-
if (!options?.skipUndoRedo && !skipEdgeRecording.current) {
995+
if (!options?.skipUndoRedo) {
1004996
edges.forEach((edge) => undoRedo.recordAddEdge(edge.id))
1005997
}
1006998

0 commit comments

Comments
 (0)