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

Update package.json - Svelte peer dependency, add v4 #51

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

0gust1
Copy link
Contributor

@0gust1 0gust1 commented Jul 6, 2023

fixes #52

A proposal to support the recent release of Svelte V4.

It was checked locally on our SvelteKit project two different production-used sveltekit-based projects, no problems so far.

We may eventually check and add the |global modifier on the transitions used in src/lib/Banner.svelte, but IMHO, it's not necessary.

Refs:

A proposal to support the recent release of Svelte V4
@0gust1 0gust1 marked this pull request as ready for review July 6, 2023 17:09
@0gust1 0gust1 mentioned this pull request Aug 1, 2023
@0gust1
Copy link
Contributor Author

0gust1 commented Aug 22, 2023

@antony First, sorry for the tagging (I feel a little bad, forgive me if you find this rude).

Can we consider moving forward with this ? On our current project, gdpr-cookie-consent-banner (which I quite glad to use) is the last dependency preventing us from migrating to svelte v4.

@antony
Copy link
Member

antony commented Aug 22, 2023

Hey @0gust1 - absolutely. My apologies for not merging this sooner!

I get so many notifications I don't see them, so thanks for reaching out :)

@antony antony merged commit 1031207 into beyonk-group:master Aug 22, 2023
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.

svelte 4 support
2 participants