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

Update fetch() vs jQuery.ajax() list intro in “Using Fetch” #3416

Merged
merged 3 commits into from
Mar 23, 2021

Conversation

jamesnw
Copy link
Contributor

@jamesnw jamesnw commented Mar 23, 2021

The list of main differences in the fetch API was reduced from 3 items to 2 items in this PR- #2558. This changes the description of the list to match the contents of the the list.

This is a small nitpick, but will prevent others from looking for a third difference that does not exist.

The list of main differences was reduced from 3 items to 2 items in this PR- mdn#2558. This changes the description of the list to match the contents of the the list.
@jamesnw jamesnw requested a review from a team as a code owner March 23, 2021 15:33
@jamesnw jamesnw requested review from sideshowbarker and removed request for a team March 23, 2021 15:33
@sideshowbarker sideshowbarker dismissed a stale review March 23, 2021 16:53

didn’t intend to approve

Co-authored-by: Michael[tm] Smith <mike@w3.org>
@sideshowbarker sideshowbarker changed the title Update list count to match list Update fetch() vs jQuery.ajax() list intro in “Using Fetch” Mar 23, 2021
@sideshowbarker sideshowbarker merged commit 7061db9 into mdn:main Mar 23, 2021
@sideshowbarker
Copy link
Member

James, thanks much, and congrats on landing your first docs change here — welcome aboard 🎉

@jamesnw jamesnw deleted the patch-1 branch March 23, 2021 19:46
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants