-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
docs(stepfunctions): integrationPattern
update
#29015
Conversation
integrationPattern
updateintegrationPattern
update
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
integrationPattern
updateintegrationPattern
update
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.
Thanks @Rizxcviii
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
### Issue # (if applicable) Closes #21915 . (Doesn't close it, but solves a point that was mentioned here) ### Reason for this change The prop is pretty useful, but hasn't received enough documentation from my perspective. This adds documentation to the overview `README` as well it's own jsdoc. ### Description of changes Added a section for the `integrationPattern` into the README, and updated the jsdoc to include more detailed information. ### Description of how you validated changes No validation. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Issue # (if applicable)
Closes #21915 . (Doesn't close it, but solves a point that was mentioned here)
Reason for this change
The prop is pretty useful, but hasn't received enough documentation from my perspective. This adds documentation to the overview
README
as well it's own jsdoc.Description of changes
Added a section for the
integrationPattern
into the README, and updated the jsdoc to include more detailed information.Description of how you validated changes
No validation.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license