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

Update http.md #5437

Merged
merged 1 commit into from Dec 31, 2018

Conversation

Projects
None yet
3 participants
@devWebNuts
Copy link
Contributor

devWebNuts commented Nov 2, 2018

The values for the events properties must be in lowercase. The display values will appear in Capital letters in Azure Portal.

What did you implement:

Change the value for the event property methods from GET to get as the properties values must be in lowercase, otherwise the binding property won't be set.

How did you implement it:

Replace GET with get

How can we verify it:

If you deploy using GET in the serverless file the methods property in the bindings for the httpTrigger will not be set to GET.
If deploying using get instead, methods property will be set as expected.

Update http.md
The values for the events properties must be in lowercase. The display values will appear in Capital letters in the Portal.
@horike37
Copy link
Member

horike37 left a comment

Good catch @devWebNuts 👍

@horike37 horike37 merged commit efd52c9 into serverless:master Dec 31, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 91.103%
Details

@dschep dschep referenced this pull request Jan 9, 2019

Merged

v1.36.0 release! #5670

@shortjared shortjared added this to the 1.36.0 milestone Jan 9, 2019

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