Remove json() utility #12146
Remove json() utility #12146
Conversation
🦋 Changeset detectedLatest commit: 73e49de The changes in this PR will be included in the next version bump. This PR includes changesets to release 10 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| typeof value.headers === "object" && | ||
| typeof value.body !== "undefined" | ||
| ); | ||
| } |
There was a problem hiding this comment.
We had this duped in @remix-run/router and remix-run/server-runtime so we can share the router one now
| export const json: JsonFunction = (data, init = {}) => { | ||
| let responseInit = typeof init === "number" ? { status: init } : init; | ||
|
|
||
| let headers = new Headers(responseInit.headers); | ||
| if (!headers.has("Content-Type")) { | ||
| headers.set("Content-Type", "application/json; charset=utf-8"); | ||
| } | ||
|
|
||
| return new Response(JSON.stringify(data), { | ||
| ...responseInit, | ||
| headers, | ||
| }); | ||
| }; |
| @@ -1,81 +0,0 @@ | |||
| import { | |||
There was a problem hiding this comment.
Everything in here was duped in the router so we use those now
brophdawg11
force-pushed
the
brophdawg11/remove-json
branch
2 times, most recently
from
540e608 to
50c8eb5
Compare
brophdawg11
force-pushed
the
brophdawg11/remove-json
branch
from
50c8eb5 to
73e49de
Compare
|
🤖 Hello there, We just published version Thanks! |
|
🤖 Hello there, We just published version Thanks! |
|
🤖 Hello there, We just published version Thanks! |
|
🤖 Hello there, We just published version Thanks! |
This was deprecated with Single Fetch in Remix v2, and folks can always use
Response.json()if needed.TODO:
types.tsissue mentioned in https://github.com/remix-run/react-router/pull/12146/files#r1805311358