-
Notifications
You must be signed in to change notification settings - Fork 1k
New serverless pattern - lambda-endusermessaging #2800
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
base: main
Are you sure you want to change the base?
New serverless pattern - lambda-endusermessaging #2800
Conversation
Any update on this team? Thank you! |
Could you please remove the folder with the bedrock example |
Thanks Ben, removed now |
actions=[ | ||
"sms-voice:SendTextMessage", | ||
], | ||
resources=["*"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please restrict this at least to the current account and region
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This wont work as its a known issue
# Create SMS Sender Lambda Function | ||
sms_sender_function = _lambda.Function( | ||
self, "SmsSenderFunction", | ||
runtime=_lambda.Runtime.PYTHON_3_11, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use most recent runtime
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
lambda-endusermessaging/README.md
Outdated
|
||
* AWS Lambda | ||
* AWS End User Messaging SMS | ||
* AWS Secret Manager |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* AWS Secret Manager | |
* AWS Secrets Manager |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
"description": "Scheduled SMS messaging using AWS Lambda and End User Messaging", | ||
"language": "Python", | ||
"level": "300", | ||
"framework": "CDK", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"framework": "CDK", | |
"framework": "AWS CDK", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Complete
@@ -0,0 +1,60 @@ | |||
{ | |||
"title": "Lambda - End User Messaging", | |||
"description": "Scheduled SMS messaging using AWS Lambda and End User Messaging", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"description": "Scheduled SMS messaging using AWS Lambda and End User Messaging", | |
"description": "Scheduled SMS messaging using AWS Lambda and AWS End User Messaging", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Complete
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted
lambda-endusermessaging/src/app.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the purpose of this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted
@@ -0,0 +1,60 @@ | |||
{ | |||
"title": "Lambda - End User Messaging", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issue #, if available:
Description of changes:
Scheduled SMS messaging using AWS Lambda and End User Messaging
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.