Re: [whatwg/fetch] Add `"json"` destination, for JSON modules (PR #1691)

@annevk commented on this pull request.

Looks great modulo some nits.

I think we can count Chromium and WebKit as supportive based on statements in https://github.com/whatwg/html/issues/7233, but I'll tag some people here just in case.

cc @Constellation @syg @hiroshige-g

> +      <dt>"<code>json</code>"
+      <dd>`<code>application/json,*/*;q=0.5</code>`

Single-space indentation.

> @@ -1725,6 +1725,7 @@ the empty string,
 "<code>frame</code>",
 "<code>iframe</code>",
 "<code>image</code>",
+"<code>json</code>",

You'll need to update `destination-table` below as well.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1691#pullrequestreview-1545494437
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fetch/pull/1691/review/1545494437@github.com>

Received on Tuesday, 25 July 2023 13:37:06 UTC