Skip to content
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

adding event for SES #176

Closed
wants to merge 4 commits into from
Closed

adding event for SES #176

wants to merge 4 commits into from

Conversation

jeromevdl
Copy link
Contributor

Issue #, if available:

Description of changes: Adding the Event for SES.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jeromevdl
Copy link
Contributor Author

Actually, it does not work because of the "Records" (upper case R), will need a mixin...

@jeromevdl
Copy link
Contributor Author

Added the mixin to serialization module but need to be ported to Lambda Runtime...

@msailes msailes added this to Require Lambda Change in Events Library Mar 18, 2021
@mr-smithers-excellent
Copy link

@msailes - would love to see an SESEvent added to the library. Currently using custom objects to deserialize. Any chance you can give this PR another look?

@msailes
Copy link
Collaborator

msailes commented Dec 17, 2021

Hi Sean,

Thanks for the feedback. We're in the run up to the next major version v4. This will make adding events like this one a lot easier.

Thanks for your patience,

Mark

@msailes
Copy link
Collaborator

msailes commented Jun 3, 2022

Closing, superseded by #345

@msailes msailes closed this Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Events Library
Require Lambda Change
Development

Successfully merging this pull request may close these issues.

None yet

3 participants