Skip to content

Conversation

@MickelSierra85
Copy link

I would recommend adding the method to make the ajax request clearer

I would recommend adding the method to make the ajax request clearer
@jquerybot
Copy link

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

@dmethvin
Copy link
Member

The JSONP transport uses a script tag, so it always performs a GET. Actually showing that in the example may actually confuse things more and make people think other methods like POST are possible when they are not. Perhaps the text could be clarified elsewhere?

@arthurvr
Copy link
Member

Can you please also take a look at the CLA error? :)

@MickelSierra85
Copy link
Author

Hi,
The CLA error should be fixed now.

@dmethvin You are right. I can remove this PR if you think its not needed

Just signed the CLA
@dmethvin
Copy link
Member

dmethvin commented Jan 5, 2016

It doesn't look needed to me. I'm not sure how alive YQL is these days either, their forum has gone missing and the last activity on the YQL blog is 2013. In this industry it's hard to find stuff that doesn't break after a few years!

According to the error message (click Details), it looks like the name on the commit still isn't right. you need to change your git info on your computer, see the instructions in Details. If we aren't going to land it though, that's fine. If you see something else that needs fixing and have fixed your git signature, everything should be fine with subsequent commits.

Base automatically changed from master to main March 8, 2021 17:15
@Krinkle Krinkle closed this Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants