Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Single Fetch: fix issues returning/throwing response stubs from resource routes #9488

Merged
merged 2 commits into from
May 28, 2024

Conversation

brophdawg11
Copy link
Contributor

No description provided.

Copy link

changeset-bot bot commented May 23, 2024

🦋 Changeset detected

Latest commit: a70bd6b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages
Name Type
@remix-run/server-runtime Patch
@remix-run/cloudflare Patch
@remix-run/deno Patch
@remix-run/dev Patch
@remix-run/node Patch
@remix-run/react Patch
@remix-run/testing Patch
@remix-run/cloudflare-pages Patch
@remix-run/cloudflare-workers Patch
@remix-run/architect Patch
@remix-run/express Patch
@remix-run/serve Patch
create-remix Patch
remix Patch
@remix-run/css-bundle Patch
@remix-run/eslint-config Patch

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

@@ -591,7 +591,7 @@ async function handleResourceRequest(
: null),
});

if (typeof response === "object") {
if (typeof response === "object" && response !== null) {
invariant(
!(DEFERRED_SYMBOL in response),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoids cannot search for property in null error

@brophdawg11 brophdawg11 changed the title fix issues returning/throwing response stubs from resource routes Single Fetch: fix issues returning/throwing response stubs from resource routes May 23, 2024
@brophdawg11 brophdawg11 merged commit fe578e0 into dev May 28, 2024
5 checks passed
@brophdawg11 brophdawg11 deleted the brophdawg11/single-fetch-fixes branch May 28, 2024 14:54
@github-actions github-actions bot added the awaiting release This issue has been fixed and will be released soon label May 28, 2024
Copy link
Contributor

🤖 Hello there,

We just published version 2.10.0-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

Copy link
Contributor

🤖 Hello there,

We just published version 2.10.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

@github-actions github-actions bot removed the awaiting release This issue has been fixed and will be released soon label Jun 25, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Single Fetch: returning null from resource route loader leads to error
2 participants