Skip to content

Conversation

tachyonics
Copy link
Contributor

Description

Moving over PR from samples repository.

Testing

Deployed lambda to AWS account
Verified lambda succeeded and expected logs were produced.

@swift-server-bot
Copy link

Can one of the admins verify this patch?

3 similar comments
@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@swift-server-bot
Copy link

Can one of the admins verify this patch?

@tomerd
Copy link
Contributor

tomerd commented May 29, 2020

@swift-server-bot add to whitelist

@tomerd
Copy link
Contributor

tomerd commented May 29, 2020

thank you @tachyonics, needs a rebase (sorry)

also the sanity script is failing on some formatting issues, you should be able to run it locally or in docker

@weissi
Copy link
Contributor

weissi commented May 30, 2020

CC @oc243

@eneko
Copy link
Contributor

eneko commented Jun 6, 2020

This is pretty cool, @tachyonics, I'm quite excited about this. I'm not a contributor to this library, but wanted to give this a try and provide some feedback.

I pulled this PR and have been running some tests, trying to learn how this works. I managed to get everything working with sam-deploy.sh, I can run the lambda and see data populated on DynamoDB. Pretty straight-forward.

However, I haven't been able to get it to work with deploy.sh. I manually configured AIM roles and permissions, but seems like I'm missing something. It woukld be great if README was updated with set up instructions.

Thank you!

@eneko
Copy link
Contributor

eneko commented Jun 6, 2020

Just got it working with deploy.sh, all I needed was to properly configure environment variables for the DynamoDB table I had granted permissions. Hoping this gets accepted (though more documentation would be great)

@tomerd
Copy link
Contributor

tomerd commented Aug 6, 2020

hi @tachyonics how do you want to proceed with this PR? note some thoughts about such additional examples in https://forums.swift.org/t/examples-and-getting-started-guides/

@tomerd
Copy link
Contributor

tomerd commented Oct 5, 2020

@tachyonics if you want to keep this around, could you please update this PR to be against the main branch so we can delete the old master branch

@tomerd tomerd closed this Oct 5, 2020
@tachyonics tachyonics deleted the dynamodb_put_backend_example branch May 9, 2022 06:46
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.

5 participants