Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

feat: add ses identity notification #1118

Closed
wants to merge 4 commits into from
Closed

Conversation

yyaskriloff
Copy link

@yyaskriloff yyaskriloff commented Sep 22, 2024

Assuming most projects are going to production setting up identity notifications is needed to mange complaints and bounce.

I tried to match the coding style if theres anything you need me to change just lmk and i'll try to do it as soon as i have time.

@yyaskriloff yyaskriloff changed the title feat: add identity notification feat: add ses identity notification Sep 22, 2024
@jayair
Copy link
Contributor

jayair commented Sep 27, 2024

Oh interesting. Can you open an issue for this?

I also think we might want to take the ARN here instead of the ARN @fwang

@fwang
Copy link
Contributor

fwang commented Oct 16, 2024

Thanks @yyaskriloff!

Event destinations support more events types and destinations, and is preferred over identity notifications. I re-implemented what you had w/ event destinations here ce169d8

Will be in the next release.

^ feel free to open an issue if this doesn't work for you.

@fwang fwang closed this Oct 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants