-
Notifications
You must be signed in to change notification settings - Fork 33
Basic support for topics #8
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
Conversation
Conflicts: package.json
…pped SNS_KEY_ID for SNS_ACCESS_KEY and vice versa in tests
This looks great @abiskop. I've been busy with work the last few days but should get around to merging this tonight. |
@abiskop could you possibly update the README with information on the newly added features? |
Sure! Will do that tomorrow! EDIT: Did not get to it so far due to stuff@work, but hopefully during the next days. |
Thanks for the update @abiskop. I can relate :) |
Alright, README update done! |
This is an awesome PR thanks @abiskop! I'll get it published to npm within 24 hours. 👏 |
Add support for topics. Update to newer AWS SDK.
@abiskop published to npm registry as version 1.2.0 |
Great, thanks! I might also add the remaining topic-/subscription-related methods like "getTopicAttributes" and stuff at some point; I just didn't really need it at this point, that's why I skipped it during the first iteration. |
Add things only when we need them. The evolutionary development process ;) |
👍 |
fixes #7