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

Uses Firebase 9 and React 18. #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

dvdpena
Copy link

@dvdpena dvdpena commented Apr 14, 2022

Firebase changed a lot in version 9, and the original tutorial uses ]Firebase 7. To use the exact same sintaxis, it's necessary to import the compat version when using Firebase 9.

To avoid this, I made the changes necessary to the sintaxis to allow the modularity in the firebase library.

If this version could be added, alongside the original, I think it would help people following the tutorial in 2022+

David E. Peña added 2 commits April 14, 2022 13:33
@yosuanicolaus
Copy link

Thank you very much for your work!

@pikapower9080
Copy link

Thank you, your code helped me migrate my fork of the app, you're the best!

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.

3 participants