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 firebase.js(lib) #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

update firebase.js(lib) #12

wants to merge 1 commit into from

Conversation

Anurag-Spec
Copy link

Due to the firebase V 9.0 release, there has been a change in how imports work. This provides a fix without having to change a lot of the stuff.

Due to the firebase V 9.0 release, there has been a change in how imports work. This provides a fix without having to change a lot of the stuff.
@b-hexsoul
Copy link

+1 this took me some time to figure out. Would be good to merge this PR

@karlhadwen
Copy link
Owner

Does this need a package.json update also?

@b-hexsoul
Copy link

@karlhadwen Yes, anyone following the course now that does npm i firebase will currently get V ^9.0.2 which is what requires this change

@b-hexsoul
Copy link

Love the tutorial btw!

@karlhadwen
Copy link
Owner

@b-hexsoul ty! would you mind bumping the package.json as well?

@Anurag-Spec
Copy link
Author

@karlhadwen I have made the changes to package.json as well and they are in a new pull request.

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