Navigation Menu

Skip to content
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

Add support for Apigatewayv2 payload 2.0 #1092

Merged
merged 5 commits into from Sep 23, 2020

Conversation

NicolasSeiler
Copy link
Contributor

src/events/http/HttpServer.js Outdated Show resolved Hide resolved
@dherault
Copy link
Owner

dherault commented Sep 7, 2020

I've come to understand that this is not a breaking change, is that correct?
Thanks a lot, will merge asap

@natac13
Copy link

natac13 commented Sep 15, 2020

Hey, thank you so much for this PR. I was really having some trouble with the API v1 vs v2 payloads.
Before this was merged I forked the master branch and applied these changes locally so that I can tests things out using the instruction in the contributing.md file.
However, the Graphql Playground does not seems to work even those the serverless lambda is up and running and my client application can connect fine.

The playground shows up but no Docs, Schema and say it received a 404 response.

Hopefully this was just something I did wrong and wont be an issue one this is merged. I just wanted to mention something just in case.

Again thank you for this PR and hopefully it can be merged soon 🤞

@dherault dherault merged commit 5060bda into dherault:master Sep 23, 2020
@dherault
Copy link
Owner

Thanks @NicolasSeiler, v6.8.0

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.

None yet

4 participants