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

Sveltekit 2 migration! #76

Merged
merged 2 commits into from
May 22, 2024
Merged

Sveltekit 2 migration! #76

merged 2 commits into from
May 22, 2024

Conversation

scosman
Copy link
Contributor

@scosman scosman commented May 22, 2024

This is just running npx svelte-migrate@latest sveltekit-2 and fixing package conflicts.

Next step: run through rest of https://kit.svelte.dev/docs/migrating-to-sveltekit-2

This is just running `npx svelte-migrate@latest sveltekit-2` and fixing package conflicts.

Next step: run through rest of https://kit.svelte.dev/docs/migrating-to-sveltekit-2
@scosman
Copy link
Contributor Author

scosman commented May 22, 2024

Everything seems to be working, but now also seeing the Using the user object as returned from supabase.auth.getSession() or from some supabase.auth.onAuthStateChange() events could be insecure! issue reported in the other PR we have pending from @kizivat : #63

Blocked on: supabase/auth-js#888

@scosman
Copy link
Contributor Author

scosman commented May 22, 2024

Fixes #22

Copy link

cloudflare-workers-and-pages bot commented May 22, 2024

Deploying saasstarter with  Cloudflare Pages  Cloudflare Pages

Latest commit: 338791f
Status: ✅  Deploy successful!
Preview URL: https://99168411.saasstarter.pages.dev
Branch Preview URL: https://sveltekit-2.saasstarter.pages.dev

View logs

@scosman scosman merged commit b3666d3 into main May 22, 2024
11 checks passed
@scosman scosman deleted the sveltekit_2 branch May 22, 2024 16:27
@itsmikesharescode
Copy link

nice ❤️

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