Skip to content
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

Add support for Svelte 4 #4996

Merged
merged 1 commit into from
Aug 14, 2023
Merged

Add support for Svelte 4 #4996

merged 1 commit into from
Aug 14, 2023

Conversation

itssumitrai
Copy link
Contributor

Svelte 4 was announced back in June 2023
https://svelte.dev/blog/svelte-4

But Tanstack table doesn't allow version 4.x.x which is preventing all of us Svelte table users to use Svelte 4 which has been out for a month now. Svelte 4 doesn't have any breaking changes (very minor ones related to custom elements), so I don't expect anything to break for Tanstack, and it should unblock people from using Svelte 4.

There is no bug for this, but there is a discussion thread

#4944

@Rar9
Copy link

Rar9 commented Jul 27, 2023

+1 please update

1 similar comment
@sukeshpabolu
Copy link

+1 please update

@itssumitrai
Copy link
Contributor Author

Hi, can any maintainer merge this change ? Its been sitting for 2 weeks 🙏 cc @tannerlinsley

@Zireael26
Copy link

Hey @tannerlinsley could you please look at the PR?
The change is super minimal. Would be very grateful for Svelte 4 support.

@tannerlinsley tannerlinsley merged commit 76383e2 into TanStack:main Aug 14, 2023
1 check passed
@tannerlinsley
Copy link
Collaborator

Done :)

@DeadOce4n
Copy link

DeadOce4n commented Aug 15, 2023

I came across this when following one of the examples from the docs (svelte basic), is it related to the version bump? Seems to work just fine, but my TS is yelling at me:

imagen

Previously I wasn't using @tanstack/svelte-table, but I noticed this was merged and wanted to try it out.

@itssumitrai itssumitrai deleted the svelte4 branch August 18, 2023 21:43
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.

None yet

6 participants