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 sveltekit-adapter-firebase #331

Merged
merged 4 commits into from
Dec 16, 2023

Conversation

simonnepomuk
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Nov 2, 2022

Deploy Preview for sveltesociety-preview ready!

Name Link
🔨 Latest commit b11a22d
🔍 Latest deploy log https://app.netlify.com/sites/sveltesociety-preview/deploys/657e2799ff7a5500083e7262
😎 Deploy Preview https://deploy-preview-331--sveltesociety-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@benmccann
Copy link
Collaborator

What's the difference between this package and the existing https://github.com/jthegedus/svelte-adapter-firebase package?

@simonnepomuk
Copy link
Contributor Author

What's the difference between this package and the existing https://github.com/jthegedus/svelte-adapter-firebase package?

The adapter you linked is not compatible with the current version of SvelteKit. Also it requires more configuration effort and is harder to set up.

@MacFJA MacFJA added the need rebase The PR need to be rebased label Feb 9, 2023
@benmccann benmccann changed the base branch from staging to main July 9, 2023 03:07
@MacFJA MacFJA added the need formatting The PR need to be formatted label Dec 16, 2023
@lachlancollins lachlancollins changed the title Add firebase adapter for sveltekit Add sveltekit-adapter-firebase Dec 16, 2023
@lachlancollins lachlancollins removed need rebase The PR need to be rebased need formatting The PR need to be formatted labels Dec 16, 2023
@lachlancollins lachlancollins merged commit c10d8f8 into svelte-society:main Dec 16, 2023
5 checks passed
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.

4 participants