-
Notifications
You must be signed in to change notification settings - Fork 24
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
Start support for Svelte 5 #78
Comments
wondering the same thing! |
Did some testing the other day and got it working with Edit: Misread this. In terms of changes for Svelte 5 compat, not using Svelte 5 here, I patched the latest version in my project w/ PNPM, adding |
It would be great to resolve this. Is there any news on supporting Svelte 5 yet? |
Svelte 5 is officially out now. Would be great for it to be supported. |
Feel free to contribute / comment on my PR: #145 |
Here's a drop-in replacement package with the changes from my pull request until official support is implemented: https://www.npmjs.com/package/@eirikk/portabletext-2-svelte-5 Follow the instructions in the readme to avoid having to rewrite your import statements. Also: use at your own risk!! All the existing tests are passing, but I can not guarantee that it's production ready. |
I see a pull request that implements basic Svelte 5 compatibility for this package, but it's been waiting approval to be merged into main for a while. Is anyone who's able to approve it checking this repo? 😅 |
Has there been any progress made to support Svelte 5?
The text was updated successfully, but these errors were encountered: