-
Notifications
You must be signed in to change notification settings - Fork 48
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
Async steps body #13
Comments
Could you attach the problem code here? |
Hi, I have successfully tested this class HelloWorld extends workflow_es.StepBody {
async run(context) {
console.log("Hello World");
await doSomethingAsync();
return workflow_es.ExecutionResult.next();
}
} |
@danielgerlag between steps, is a step will waiting for the previous step finished with async? |
yes it does, but if you call an async method or promise from within the step and don't |
so, it can not await for async method |
as you can see here: https://github.com/danielgerlag/workflow-es/blob/master/core/src/services/workflow-executor.ts#L73 it does in fact await, but if you fail to await any nested async methods inside your step implementation, then it's your choice |
Hi,
How to implement async step body... the system crashes when i add await to step body....
The text was updated successfully, but these errors were encountered: