-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
feat(ses-actions): move SES rule actions to separate package #4163
Conversation
Thanks so much for taking the time to contribute to the AWS CDK ❤️ We will shortly assign someone to review this pull request and help get it
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Move cross-service integrations for receipt rule actions to their own package in accordance with the CDK guidelines. Closes aws#3726 BREAKING CHANGE: adding an action to a receipt rule now requires an integration object from the `@aws-cdk/aws-ses-actions` package.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
packages/@aws-cdk/aws-ses/README.md
Outdated
``` | ||
When using `addRule`, the new rule is added after the last added rule unless `after` is specified. | ||
|
||
[More actions](test/integ.receipt.ts) | ||
Actions can be found in the `@aws-cdk/aws-ses-actions` package. |
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.
awesome to have examples get updated! feels like this should come ahead of code samples that use actions though.
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.
What do you mean here exactly, can you clarify? The integ tests for actions are now in aws-ses-actions
and you want to point to examples there?
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.
yup, the snippets of code that reference actions don't include the import of the actions package, so it'd be good to move this before the code samples :)
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@shivlaks are you sure |
@jogold good call, let me check and take care of it. temporarily added a do-not-merge so mergify doesn't surprise us in an unexpected way :) |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Pull request has been modified.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Move cross-service integrations for receipt rule actions to their own package in accordance with
the CDK guidelines.
Closes #3726
BREAKING CHANGE: adding an action to a receipt rule now requires an integration
object from the
@aws-cdk/aws-ses-actions
package.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license