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

Feature/sns notifications #63

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

oma
Copy link

@oma oma commented Aug 11, 2016

SNS: SES Identity Notifications:

  • get identity notifications (SNS topics for SES bounces etc)
  • register identity notifications in topics Bounce | Complaint | Delivery
  • delete identity notifications
  • unit tests
  • production tested
  • list multiple identities notifications (don't find it useful although API supports it)

Identity

  • list email identities
  • list domain identities

This commit is cherry-picked and modified to not touch version or gemspec.
Note: I cannot build the gem, generate gemspec or generate rdoc. Jeweler not working for me and I think you'd rather control this yourself.

@oma
Copy link
Author

oma commented Aug 12, 2016

@drewblas wdyt?

@drewblas
Copy link
Owner

Looks good. Give me a day or two to do some testing and then I'll merge and release

@oma
Copy link
Author

oma commented Oct 26, 2016

any update on this ? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants