-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(core): Remove unused execution status warning
(no-changelog)
#9759
Conversation
@@ -36,7 +36,7 @@ interface IExecutionError { | |||
|
|||
export interface IWorkflowExecutionProgress { | |||
workflowId: string; | |||
status: ExecutionStatus; | |||
status: ExecutionStatus | 'warning'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here only used for reporting, not persisted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should use another type, extend IWorkflowExecutionProgress
to something like CliExecutionResult
that could contain warning
as one of the status, how does it sound? Maybe outside of the scope of this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, for now I'll keep this PR about simplifying ExecutionStatus
. Let's refine types for CLI commands in future.
@@ -36,7 +36,7 @@ interface IExecutionError { | |||
|
|||
export interface IWorkflowExecutionProgress { | |||
workflowId: string; | |||
status: ExecutionStatus; | |||
status: ExecutionStatus | 'warning'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should use another type, extend IWorkflowExecutionProgress
to something like CliExecutionResult
that could contain warning
as one of the status, how does it sound? Maybe outside of the scope of this PR.
✅ All Cypress E2E specs passed |
4 flaky tests on run #5513 ↗︎Details:
|
Test | Artifacts | |
---|---|---|
NDV > should not retrieve remote options when required params throw errors |
Screenshots
Video
|
|
NDV > Stop listening for trigger event from NDV |
Screenshots
Video
|
10-undo-redo.cy.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
Undo/Redo > should undo/redo adding connected nodes |
Test Replay
Screenshots
Video
|
20-workflow-executions.cy.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
Current Workflow Executions > should auto load more items if there is space and auto scroll |
Test Replay
Screenshots
Video
|
Review all test suite changes for PR #9759 ↗︎
Got released with |
1 similar comment
Got released with |
https://n8nio.slack.com/archives/C069HS026UF/p1706085281530329