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

apigateway: "methodResponses" is missing from MethodOptions #905

Closed
eladb opened this issue Oct 11, 2018 · 6 comments · Fixed by #2960 or MechanicalRock/tech-radar#14 · May be fixed by MechanicalRock/cdk-constructs#5, MechanicalRock/cdk-constructs#6 or MechanicalRock/cdk-constructs#7
Labels
@aws-cdk/aws-apigateway Related to Amazon API Gateway feature-request A feature should be added or improved.

Comments

@eladb
Copy link
Contributor

eladb commented Oct 11, 2018

For some reason, methodResponses is missing from the API Gateway Method construct.

@eladb eladb added the feature label Oct 14, 2018
@rix0rrr rix0rrr added feature-request A feature should be added or improved. and removed feature-request A feature should be added or improved. feature labels Nov 6, 2018
@debora-ito debora-ito added the @aws-cdk/aws-apigateway Related to Amazon API Gateway label Nov 7, 2018
@john-shaskin
Copy link
Contributor

Is anyone working on this? I was thinking of picking this up.

@eladb
Copy link
Contributor Author

eladb commented Dec 23, 2018

@john-shaskin go ahead, it's all yours! ❤️

@rix0rrr rix0rrr added the gap label Jan 4, 2019
@john-shaskin
Copy link
Contributor

Getting a PR together for this shortly.

@john-shaskin
Copy link
Contributor

Addressed in #1572

@divyachaudhary2
Copy link

Hi is there any update on when this will be deployed?

@john-shaskin
Copy link
Contributor

The PR is awaiting a second review. I’ll do some prodding.

eladb pushed a commit that referenced this issue Jun 27, 2019
…d validators (#2960)

Fixes #905: "apigateway: "methodResponses" is missing from MethodOptions"
Fixes #1695: apigateway: missing support for models
Fixes #727: API Gateway: improve API for request parameters and responses
Fixes #723: API Gateway: missing features
Fixes #2957: RestApi to use logical id as a name for APIs instead of name of current construct
Adds support for JsonSchema in Model
Aligns Model to the PhysicalName convention.
No breaking change, documentation updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment