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

Added support to have an alias subscribe to an existing SNS topic #94 #95

Merged
merged 4 commits into from
Jan 25, 2018

Conversation

zachboyd
Copy link
Contributor

Closes #94

This PR adds support for resources of type AWS::SNS::Subscription to be transferred to the alias stack. This resource will be present in the cloudformation template when a lambda subscribes to an existing SNS topic by ARN.

@HyperBrain
Copy link
Member

Hi @zachboyd , thanks for the contribution 💯 .

I will review it as soon as possible.

@zachboyd
Copy link
Contributor Author

Thanks! I just fixed the lint issues and pushed changes as well.

@HyperBrain
Copy link
Member

BTW: The project contains a proper .editorconfig configuration file, so, if you use an editorconfig plugin in your editor, it will be automatically correct 😄

@zachboyd
Copy link
Contributor Author

Great! I went ahead and installed the plugin for any future work I am able to contribute back.

@baipi
Copy link

baipi commented Jan 25, 2018

Thanks @zachboyd, it's exactly what I was looking for and it works well! 👍

@HyperBrain
Copy link
Member

Merging. Thanks @zachboyd for the implementation and @baipi for testing 💯

@HyperBrain HyperBrain merged commit 87b12a3 into serverless-heaven:master Jan 25, 2018
defnotrobbie pushed a commit to 1stdibs/serverless-aws-alias that referenced this pull request Nov 14, 2022
…n-fix

Added support to have an alias subscribe to an existing SNS topic serverless-heaven#94
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.

3 participants