-
Notifications
You must be signed in to change notification settings - Fork 1
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
Working through learn-aws-lambda #13
Comments
Our time estimate has been way off. We expected this to take 1 day but it is definitely going to take longer. We're currently around 1/4 of the way through due to doing a lot of independent research. Hopefully 2 days will be enough. |
@ZooeyMiller when ever you think an estimate is going to take longer than |
The aws API is not my favourite... It seems like most of the code in this repo is out of date so most sections will need a fair bit of updating. |
@nelsonic apologies, I misinterpreted the |
@finnhodgkin if a lot is out-of-date, please only update the bits that are relevant to:
For context, we are going to be working on this: https://github.com/dwyl/aws-ses-lambda soon hence wanting to update the Lambda-related knowledge. 😉 |
'A fair bit' was probs an exaggeration. The menus have changed a little, but not so much that we should re-write the whole thing. The main change is in JS syntax - context.succeed() has been replaced by error first callbacks. |
@finnhodgkin yeah, the callback syntax is preferable but both are still valid. 😉 |
@nelsonic oh that's strange- I was getting errors with context.succeed(). I'll check again when I get in to make sure it's not some other error, but the aws docs do say:
There's actually already a paragraph in there about callbacks vs context but it's quite far down the page. Might make more sense to move it up to the top. |
Update: context.done/succeed definitely do still work so it must have been some other error. Still worth converting all the snippets to callbacks at some point given the above recommendation from Amazon. |
I'm currently researching how to call a lambda when SES receives mail. There's nothing in learn-aws-lambda about SES so it's all new info 👍 Info here: dwyl/learn-aws-lambda#97 |
@nelsonic with regards to dwyl/learn-aws-lambda#97 (comment) what should @finnhodgkin and I work on in the meantime? |
While we wait for a response we'll go through |
We've done the thing where an email triggers a lambda function, but we're not 💯% clear that it's actually running. To test it we're going to make the lambda save something to S3, and then if that works we'll get the email body. 👍 |
I've recently used the guid to deploy a Lambda: nelsonic/github-scraper#128 🚀 |
@finnhodgkin and I will be working through
learn-aws-lambda
today, this morning we have been working on issues inlearn-amazon-web-services
with time estimates in the issues on that repo.I am estimating a day for
learn-aws-lambda
as it looks like quite a long repo. Any issues we raise will be individually time estimated as well and those time estimates are not necessarily contained in the day assigned to this issue.EDIT: Due to overrunning the day time estimate, each of these chapters in
learn-aws-lambda
(apart from the ones already completed) will be made as individual issues and time estimated on their own.Trigger a Lambda function using an event from DynamoDBTrigger a Lambda function using the Simple Notification SystemContinuous Integration using CodeshipTesting Lambda FunctionsUpload Lambda Function to S3 and deploy to LambdaDeploying Lambda Functions using GulpVersioning and Aliasing Lambda FunctionsCreate an API with GET/POST Methods using Lambda functions to retrieve/update records from a DynamoDB tableWe've struck through these parts as they are not relevant to the things mentioned in #13 (comment) so today will be working on:
Neither of these are already in the repo so we will have to make them from scratch.
The text was updated successfully, but these errors were encountered: