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 "deadLetter" to the list of valid function properties. #44

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

added "deadLetter" to the list of valid function properties. #44

wants to merge 2 commits into from

Conversation

amilliken-bb
Copy link

This is to fix the following error:
Configuration error at 'functions.<functionName>': unrecognized property 'deadLetter'

@rachnareddyg
Copy link

I am also getting the configuration warning message:
Serverless: Configuration warning at 'functions.main': unrecognized property 'deadLetter'

Can this pull request be merged?

Copy link

@rachnareddyg rachnareddyg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dxd1 added a commit to epsyhealth/serverless-plugin-lambda-dead-letter that referenced this pull request Mar 31, 2022
ivantoniev added a commit to BloomAndWild/serverless-plugin-lambda-dead-letter-v2-compat that referenced this pull request Feb 17, 2023
We want to upgrade to a newer serverless version, to do that, we need to
make this change to work with the new serverless dsl.

This was taken from the original repo which is no longer maintained and
the proposed fix was not merged.

gmetzker#44
@ydinari
Copy link

ydinari commented Mar 12, 2023

Hi, why isn't this PR merged?
Thanks

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.

None yet

3 participants