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

Body.json with reviver #188

Closed
wants to merge 2 commits into from
Closed

Body.json with reviver #188

wants to merge 2 commits into from

Conversation

heruan
Copy link

@heruan heruan commented Aug 5, 2015

Body.json() should take an optional parameter to be passed to JSON.parse(json[, reviver]) function to use a reviver callback to parse JSON data.

P.S. I understand this is a polyfill, see whatwg/fetch#104

`Body.json()` should take an optional parameter to be passed to `JSON.parse(json[, reviver])` function to use a _reviver_ callback to parse JSON data.

P.S. I understand this is a polyfill, see whatwg/fetch#104
@heruan heruan mentioned this pull request Aug 5, 2015
@heruan
Copy link
Author

heruan commented Aug 5, 2015

Such feature will not be present in the specs.

@heruan heruan closed this Aug 5, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 3, 2020
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.

1 participant