-
Notifications
You must be signed in to change notification settings - Fork 29
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
Parallel state should allow Catch property #2
Comments
Hi @mrgrain,
Which results in this: {
"StartAt": "A",
"States": {
"A": {
"Type": "Parallel",
"Next": "D",
"Branches": [{
"StartAt": "B",
"States": {
"B": {
"Type": "Fail"
}
}
},
{
"StartAt": "C",
"States": {
"C": {
"Type": "Pass",
"End": true
}
}
}
],
"Catch": [{
"ErrorEquals": [
"States.ALL"
],
"Next": "D"
}]
},
"D": {
"Type": "Fail"
}
}
} Fixed in 1.0.3. Feel free to reopen if needed |
Thanks, in the example I was trying to simplify my state machine - obviously not with much success. |
ChristopheBougere
added a commit
that referenced
this issue
Jul 7, 2022
* chore: rewrite to typescript BREAKING CHANGE: updated the json path error * fix: typescript review (remove any) * feat: integrate asl path validator (#2) * feat(paths): update to latest path validator * chore: resolve conflicts * chore: merge cleanup * refactor: extract arn formatter and added tests * fix: restored support for array * fix: restored support for cloud formation functions for resources * feat: better filtering of error messages tests: added new test for asserting error messages chore: updated tsconfig to keep js files out of src Co-authored-by: Christophe Bougère <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Given a Parallel state
It should allow the
Catch
property: http://docs.aws.amazon.com/step-functions/latest/dg/amazon-states-language-parallel-state.html?shortFooter=trueHowever it fails.
Version 1.0.2
The text was updated successfully, but these errors were encountered: