ajax: documented new post and get config object interfaces#622
ajax: documented new post and get config object interfaces#622togakangaroo wants to merge 4 commits into
Conversation
|
Unfortunately I couldn't get the vagrant VM running so this is untested but I think it's alright? |
There was a problem hiding this comment.
Missing dot at the end of your sentence. 😄
There was a problem hiding this comment.
Type will automatically be set to
POST
This seems more clear to me:
The
typeoption will automatically be set toPOST.
There was a problem hiding this comment.
Awesome that you caught the $.ajaxSetup point.
There was a problem hiding this comment.
I think we should be consistent within this paragraph, right now it's $.ajaxSetup and jQuery.ajax. I know we use both in docs but whichever is most common elsewhere should be used.
|
For the reference, issue is #620. IMHO I'd like to see an example using the new technique in here.
@togakangaroo Looks alright.
You might mean jquery/jquery#1995 ;) Thanks for the PR! It's awesome when people doing stuff in core come in here and update the docs! 😄 |
There was a problem hiding this comment.
This will end up being 3.0, that's our next release.
|
@arthurvr do you have a working docs setup? Can you help @togakangaroo by pulling down this PR and seeing if it passes XMLLint checks? |
|
@dmethvin That's what I meant with "looks alright". Everything passes and is building/deploying nicely. |
|
Alright, fixed up the points above. @dmethvin regarding |
|
Let's just be consistent here for both. Based on the |
There was a problem hiding this comment.
Looks like this breaks the indentation. Could you fix that? I'd love to land this PR soon.
There was a problem hiding this comment.
Arthur, you could edit this with an --amend IMO, it's a pretty trivial thing.
Documents changes in PR 1999