Fix not available workflow version id in runs (#13958)

Quick fix : make workflow version available in runs.
This would global refactor. We should not have both version and run flow
here.
I will probably stop using the same component in both at some point and
rather duplicate. Versions and Workflow will use version. Runs will use
flow.
This commit is contained in:
Thomas Trompette
2025-08-18 11:52:40 +02:00
committed by GitHub
parent 025b59b5d0
commit 1598e5590a
@@ -9,6 +9,7 @@ import { useWorkflowVersion } from '@/workflow/hooks/useWorkflowVersion';
import { flowComponentState } from '@/workflow/states/flowComponentState';
import { workflowVisualizerWorkflowIdComponentState } from '@/workflow/states/workflowVisualizerWorkflowIdComponentState';
import { workflowVisualizerWorkflowRunIdComponentState } from '@/workflow/states/workflowVisualizerWorkflowRunIdComponentState';
import { workflowVisualizerWorkflowVersionIdComponentState } from '@/workflow/states/workflowVisualizerWorkflowVersionIdComponentState';
import { type WorkflowRunState } from '@/workflow/types/Workflow';
import { workflowDiagramComponentState } from '@/workflow/workflow-diagram/states/workflowDiagramComponentState';
import { workflowDiagramStatusComponentState } from '@/workflow/workflow-diagram/states/workflowDiagramStatusComponentState';
@@ -32,11 +33,16 @@ export const WorkflowRunVisualizerEffect = ({
const { getIcon } = useIcons();
const workflowRun = useWorkflowRun({ workflowRunId });
const workflowVersion = useWorkflowVersion(workflowRun?.workflowVersionId);
const setWorkflowRunId = useSetRecoilComponentState(
workflowVisualizerWorkflowRunIdComponentState,
);
const workflowVersionId = workflowRun?.workflowVersionId;
const workflowVersion = useWorkflowVersion(workflowVersionId);
const setWorkflowVisualizerWorkflowVersionId = useSetRecoilComponentState(
workflowVisualizerWorkflowVersionIdComponentState,
);
const workflowVisualizerWorkflowIdState = useRecoilComponentCallbackState(
workflowVisualizerWorkflowIdComponentState,
);
@@ -84,6 +90,14 @@ export const WorkflowRunVisualizerEffect = ({
setWorkflowVisualizerWorkflowId(workflowRun.workflowId);
}, [setWorkflowVisualizerWorkflowId, workflowRun]);
useEffect(() => {
if (!isDefined(workflowVersionId)) {
return;
}
setWorkflowVisualizerWorkflowVersionId(workflowVersionId);
}, [setWorkflowVisualizerWorkflowVersionId, workflowVersionId]);
const handleWorkflowRunDiagramGeneration = useRecoilCallback(
({ snapshot, set }) =>
({