-
-
Notifications
You must be signed in to change notification settings - Fork 107
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
Feature request: Svelte 4 support #302
Comments
I'll maybe open a PR for svelte 4 support when I have the time, the migration script should be doing all the heavy lifting so it'll be a quick update |
👋 I think we definitely do want svelte 4 support. I will not be able to look into this until next weekend at the earliest. But I welcome any PR! It may be a good opportunity to also go through with some breaking changes. |
created a PR here |
When will this be merged? this project is what made me to stay with Svelte, I don't like sveltekit's SSR-first approach as I only need CSR SPA for all my projects, as I can not run node.js(SSR) on my boards due to hardware resource constraints. Thanks. |
Please merge and support Svelte-4. Svelte-5 is also on its way :) |
@ItalyPaleAle any update on this? |
@mikeychowy are you still planning on working on this? |
I might look into it on Thursday or Friday to perform the @mikeychowy changes without the linter to support svelte4. @ItalyPaleAle hai tempo di darci un occhio nel weekend nel caso? |
I opened the PR #309 bumping the changes from @mikeychowy |
@ItalyPaleAle did you have time to take a look? |
Thanks for the PR! I will get it merged later today. Right now the CI is broken and I'll have to fix that first. Expect a release in about 24 hours |
Thanks @carbogninalberto for the PR. I fixed the linter issues and merged it. I am going to do some additional validation tomorrow and then cut the release |
Version 4.0.0 has been released with Svelte 4 support! |
Hey! Just wanted to ask if there is a plan to update the lib to support Svelte 4? As it stands it does not compile for Svelte 4 giving typescript errors when using wrap function. Current workaround is typecast it like so:
The text was updated successfully, but these errors were encountered: