This repository has been archived by the owner on Aug 16, 2022. It is now read-only.
We should not foster and propagate wrong concepts to programmers #16
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
the `.json()` is to parse JSON (string), and returns a JavaScript **object**. It DOES NOT return JSON. It returns data, in the form of a JavaScript object. There are programmers coming up, asking about a JavaScript object but calling it a json. `.json()` is not to return JSON. `.json()` is to parse JSON and return data, in the form of a JavaScript object. We should not foster and propagate wrong concepts to thousands and millions of programmers.
|
Anyway why this hasn't merged yet... |
|
Hi @sideshowbarker, pinging you here because you seem to have worked extensively with @KennethKinLum on the related PR mdn/content#3098. Do you think you could review this one as well and merge? |
sideshowbarker
approved these changes
May 5, 2022
|
@KennethKinLum Thanks much for catching this and fixing it. @dipikabh Thanks much for the heads-up. I don't actually have merge perms for this repo. If you do, could you please go ahead and merge it? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
the
.json()is to parse JSON (in the form of a string or text), and returns a JavaScript object.It DOES NOT return JSON. It returns data, in the form of a JavaScript object.
There are programmers coming up on StackOverflow, asking about a JavaScript object but calling it a json.
It is very important.
.json()is not to return JSON..json()is to parse JSON and return data, in the form of a JavaScript object. We should not foster and propagate wrong concepts to thousands and millions of programmers.