fix(core): Assign properties.success earlier to set executionStatus correctly

This commit is contained in:
Michael Auerswald 2023-03-24 12:32:39 +01:00 committed by कारतोफ्फेलस्क्रिप्ट™
parent 1663319c1f
commit 6c7772a0b3

View file

@ -284,6 +284,8 @@ export class InternalHooks implements IInternalHooksClass {
properties.user_id = userId;
}
properties.success = !!runData?.finished;
let executionStatus: ExecutionStatus;
if (runData?.status === 'crashed') {
executionStatus = 'crashed';
@ -295,7 +297,6 @@ export class InternalHooks implements IInternalHooksClass {
if (runData !== undefined) {
properties.execution_mode = runData.mode;
properties.success = !!runData.finished;
properties.is_manual = runData.mode === 'manual';
let nodeGraphResult: INodesGraphResult | null = null;