-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
ses_identity_notification_topic: add support for include_original_headers #7293
ses_identity_notification_topic: add support for include_original_headers #7293
Conversation
@vaskevich would you have the time to update this? If not please yell and I'll get my merge boots on :) |
10e457f
to
5db3465
Compare
Sure, updated. |
Thanks @vaskevich. Hopefully, this will get merged soon and cut out the only manual step in my proccess ⛏ |
Any updates on this? |
It's ready to go on my end. Just need a maintainer to review and merge. |
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.
Hi @vaskevich 👋 Sorry for the delay in getting this reviewed. This looks good to me! 🚀
Output from acceptance testing:
--- PASS: TestAccAwsSESIdentityNotificationTopic_basic (17.70s)
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Fixes #4376
Changes proposed in this pull request:
Output from acceptance testing: