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
AWS - SES - Create new resource type configuration-set for SES #8457
Conversation
|
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, thanks
@ajkerrigan / @thisisshi, when you get a chance, do you mind taking a look at it one more time. Code looks good to me, I also went ahead and tested the tagging functionality and can confirm that the resource supports resourcegroupstagging api. Thanks :) |
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, the core looks good to me. Suggestion inline to avoid including noise or duplicate data during augment.
Co-authored-by: AJ Kerrigan <aj@speckledmonkey.com>
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, thanks @peterheo and @ajkerrigan
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.
Creating a new resource type for the SES service.