Async awaitify code #4

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.

Sign up for GitHub

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

Jump to bottom

Open

DavidWells wants to merge 2 commits into theburningmonk:master from DavidWells:async-awaitify

DavidWells commented Jun 14, 2019

Was going through the course again and refactored co library out of most of the code in favor of async await.

Totally up to you if you want to merge 😃

The only thing I didn't touch was the tests.

Also fixed 2 missing correlationIds imports in:

manning-aws-lambda-in-motion/functions/accept-order.js

Line 19 in 5708e85

correlationIds.set('order-id', orderId);
manning-aws-lambda-in-motion/functions/fulfill-order.js

Line 19 in 5708e85

correlationIds.set('order-id', orderId);

Thanks for this awesome course

The middy wrapper pattern is super nice https://github.com/theburningmonk/manning-aws-lambda-in-motion/blob/master/middleware/wrapper.js

DavidWells added 2 commits

June 14, 2019 12:35


          Remove co and async-awaitify code

7576c5c


          remove co

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet