Skip to content

Conversation

@msakrejda
Copy link

Now that 19 is GA, this should be allowed and seems to just work.

Now that 19 is GA, this should be allowed and seems to just work.
@vercel
Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
suspense ❌ Failed (Inspect) Dec 7, 2024 10:27pm

@msakrejda
Copy link
Author

I think this warning also needs to be updated, but not sure about the wording. Thoughts?

@msakrejda
Copy link
Author

msakrejda commented Dec 6, 2024

Also, thanks for working on this library! It was really helpful in understanding the details of how Suspense works.

@msakrejda
Copy link
Author

Scratch that, I think I had the wrong version in node_modules. It does error out with 19, with

Error: unstable_getCacheForType is not a function.

It looks like maybe that API changed (and was renamed).

@msakrejda msakrejda marked this pull request as draft December 7, 2024 00:02
@bvaughn
Copy link
Owner

bvaughn commented Dec 7, 2024

The release of React 19 likely calls for a version 1.0 release of this library and dropping support for previous experimental versions of React. This is something I'll need to think about some when I get the chance. Hopefully it won't be a major change but I haven't kept up with the cache API semantics.

@msakrejda
Copy link
Author

Okay, thanks for weighing in. I'll try to get this to work with 19, and I'll update the PR, but I'm not at all familiar with these APIs so I don't know what I'm doing—I understand these are sensitive changes and they'll take a while to review.

"react": "^18.0.0",
"react-dom": "^18.0.0"
"react": "^19.0.0",
"react-dom": "^19.0.0"
Copy link
Author

Choose a reason for hiding this comment

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

I'm not sure what the best way is to support both, so for now I'm only focused on getting it to work with 19.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants