Skip to content

Conversation

@bluecurio
Copy link
Contributor

Set the HTTP method using the type attribute (instead of the method attribute), as defined in the jQuery.ajax documentation.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 95bc903 on bluecurio:issue133 into 12a65e5 on elasticsearch:2.4.

@spalger
Copy link
Contributor

spalger commented Aug 23, 2014

Thanks @bluecurio, Have you signed the CLA?

@bluecurio
Copy link
Contributor Author

Hey Spencer,

No, I haven't yet signed the CLA, but will do so tonight (if possible.)
Thanks for pointing that out -- I completely missed that part of the
documentation. (RTFM!)

Cheers,
—Daniel

-Daniel

On Sat, Aug 23, 2014 at 3:49 PM, Spencer notifications@github.com wrote:

Thanks @bluecurio https://github.com/bluecurio, Have you signed the CLA
https://github.com/elasticsearch/elasticsearch-js/blob/2.4/CONTRIBUTING.md#5-sign-the-cla
?


Reply to this email directly or view it on GitHub
#134 (comment)
.

@bluecurio
Copy link
Contributor Author

CLA signed! Let me know if there's anything else I can do. (Other than possibly submitting future pull requests 😉 )

@spalger spalger merged commit 95bc903 into elastic:2.4 Sep 4, 2014
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.

3 participants