Skip to content

Conversation

@MakendranG
Copy link
Contributor

Issue #, if available:

Description of changes:

This pattern shows how to deploy a terraform template that builds a Lambda function which calls translate for text translation.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

MakendranG and others added 9 commits February 27, 2025 20:56
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Copy link
Contributor Author

@MakendranG MakendranG left a comment

Choose a reason for hiding this comment

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

Have added the suggested changes

@MakendranG MakendranG requested a review from bfreiberg February 27, 2025 16:04
Copy link
Contributor

@bfreiberg bfreiberg left a comment

Choose a reason for hiding this comment

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

Looks good, some minor comments still left

MakendranG and others added 5 commits March 1, 2025 11:50
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
Copy link
Contributor Author

@MakendranG MakendranG left a comment

Choose a reason for hiding this comment

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

Have updated the necessary change

@MakendranG MakendranG requested a review from bfreiberg March 1, 2025 06:26
@bfreiberg
Copy link
Contributor

Looks good, thanks for your contribution. Please address that last comment and then it's good to go

Co-authored-by: Ben <9841563+bfreiberg@users.noreply.github.com>
@bfreiberg
Copy link
Contributor

Looks good, thanks for your contribution. Your PR will be merged to Serverlessland soon

@julianwood julianwood merged commit dd7e665 into aws-samples:main Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants