-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
fix(editor): Remove infinite loading in not found workflow level execution #5174
fix(editor): Remove infinite loading in not found workflow level execution #5174
Conversation
…-to-access-execution-dont
…-to-access-execution-dont
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.
LGTM; left a comment with a minor nitpic
const executions: IExecutionsSummary[] = | ||
await this.workflowsStore.loadCurrentWorkflowExecutions(this.filter); | ||
return executions; | ||
return await this.workflowsStore.loadCurrentWorkflowExecutions(this.filter); |
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.
return await this.workflowsStore.loadCurrentWorkflowExecutions(this.filter); | |
return this.workflowsStore.loadCurrentWorkflowExecutions(this.filter); |
I think the await
statement is not needed here since we return a promise anyway, right?
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.
Depending on the use case, here return await
is desired I think (usually that's the case inside a try-catch
block).
https://jakearchibald.com/2017/await-vs-return-vs-return-await/
Got released with |
* master: 📚 Update CHANGELOG.md and main package.json to 0.212.1 🔖 Release n8n@0.212.1 ⬆️ Set n8n-core@0.151.2, n8n-editor-ui@0.178.1, n8n-nodes-base@0.210.1 and n8n-workflow@0.133.2 on n8n 🔖 Release n8n-editor-ui@0.178.1 ⬆️ Set n8n-workflow@0.133.2 on n8n-editor-ui 🔖 Release n8n-nodes-base@0.210.1 ⬆️ Set n8n-core@0.151.2 and n8n-workflow@0.133.2 on n8n-nodes-base 🔖 Release n8n-node-dev@0.90.2 ⬆️ Set n8n-core@0.151.2 and n8n-workflow@0.133.2 on n8n-node-dev 🔖 Release n8n-core@0.151.2 ⬆️ Set n8n-workflow@0.133.2 on n8n-core 🔖 Release n8n-workflow@0.133.2 ci: Fix typing issues in cli tests (no-changelog) (#5227) fix(editor): Remove infinite loading in not found workflow level execution (#5174) fix: IsWeekend not checking if DateTime (#5221) (no-changelog) fix(core): Fix execute-once incoming data handling (#5211) fix(core): Make pindata with webhook responding on last node manual-only (#5223) fix(core): Fix onWorkflowPostExecute not being called (#5224) fix(core): Fix expression extension misdetection (#5219) fix: Add schema to postgres migrations (hotfix) (#5218)
No description provided.