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

Adds SQS events to zappa #1614

Closed
wants to merge 9 commits into from
Closed

Adds SQS events to zappa #1614

wants to merge 9 commits into from

Conversation

jneves
Copy link
Collaborator

@jneves jneves commented Sep 19, 2018

Fixes: #1613

Description

Adds support to SQS events.

This version uses an event source on kappa: garnaat/kappa#133 My main doubt is if we should add it to zappa for a faster deploy.

GitHub Issues

#1613

@jneves jneves closed this Sep 19, 2018
@jneves
Copy link
Collaborator Author

jneves commented Sep 19, 2018

Closed because I seem to have added some unrelated commits. Reopening soon.

@leobarcellos
Copy link

Excelent @jneves !
The unrelated commit you mean is the lines 32-34 of zappa/utilities.py ?

I'm going to try it :)

@jneves
Copy link
Collaborator Author

jneves commented Sep 21, 2018

The correct working one is in #1617 @leobarcellos

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