-
Notifications
You must be signed in to change notification settings - Fork 421
Feature request: Add CloudFormation CustomResource events to EventSource dataclasses #4332
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
Comments
Thanks for opening your first issue here! We'll come back to you as soon as we can. |
Hi @leandrodamascena, yes, I'm happy to contribute this feature. What are the next steps here? |
Hello @phipag! That's amazing to hear! Check out the steps below and let me know if you need additional help. We use Poetry as the package manager and we have a 1 - You need to create a new file called 2 - You need to export this new class here - develop/aws_lambda_powertools/utilities/data_classes/init.py 3 - You need to create a file with a fake payload here - develop/tests/events 4 - You need to write a new test for this new class - Use this test as an example - develop/tests/unit/data_classes/test_vpc_lattice_event.py 5 - You need to add this new class to the documentation - develop/docs/utilities/data_classes.md 6 - Run the tests by typing 7 - If everything is right, just open the PullRequest and have a cold drink 🍹 |
Cool. Thanks for detailed steps. That all makes sense to me. I'll reach out if something comes up on the way. |
@leandrodamascena I submitted above PR for this. |
|
This is now released under 2.39.0 version! |
Use case
I am building a Lambda handler for managing CDK CustomResources (https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.custom_resources-readme.html).
Solution/User Experience
As a user of AWS Lambda Powertools I would like to have dataclass parsing and type-safety for all standard Lambda events coming from an AWS service. CloudFormation custom resources are missing.
Alternative solutions
No response
Acknowledgment
The text was updated successfully, but these errors were encountered: