-
Notifications
You must be signed in to change notification settings - Fork 22
Updated Example #5
base: master
Are you sure you want to change the base?
Conversation
Hi @whamtet and thanks for your contributions. I don't use or maintain this project (or AWS Lambda for that matter) anymore so I can't help with this I'm afraid. |
Its ok, clojure makes it easy to do an ad hoc solution! Thanks for your On Tuesday, 8 November 2016, Ragnar Dahlén [email protected] wrote:
|
@whamtet i think that u should create another example and say that it is example of using lambda with api gateway. Because you code example can confuse someone. It is better to has 2 separate examples. Does that make sense? @ragnard if u not maintain this project, could you give contribution permission to someone? Both to accept pull request and deploy to clojars? I have time and motivation to do that. |
I decided to fork and use my own version of this library. I added validation, tests, travis etc. Please see https://github.com/Atsman/lambada |
Looking good @Astma!
…On 15 August 2017 at 06:01, Astma ***@***.***> wrote:
I decided to fork and use my own version of this library. I added
validation, tests, travis etc. Please see https://github.com/Atsman/
lambada
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AB0kP9CoaAvMKnnyUZLTYHcVzUxZKRcfks5sYMPGgaJpZM4Kqd-4>
.
|
@atsman Would you be willing to turn on issues for your fork? Github doesn't turn on issue tracking for forks by default :) |
@jamescgibson yeah sure, done |
The example works fine when you invoke it from the command line, but to get the function working through the API gateway you have to return a map with :statusCode, :headers and :body keys.