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

fix error handling #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

schnipseljagd
Copy link

@schnipseljagd schnipseljagd commented Jan 31, 2017

when e.g. dynamodb produces an error. At the moment it is silently ignored.

To test this you can just remove the IAM role statements from the serverless.yml.

when e.g. dynamodb produces an error. Before it was silently ignored.
@paolini
Copy link

paolini commented Aug 31, 2017

I also encountered this issue. Why the PR is not being considered? Is the project stalled or there is some development happening elsewhere?

@reergymerej
Copy link

I just forked so I could include my (very similar) changes. This would save other noobs a lot of time and heartache.

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