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
Firebase 9 Support #153
Firebase 9 Support #153
Conversation
There is something that have to be done to get this PR merged? |
I wanted to check up on this too! |
I forked this repo, updated dependencies, replaced 2 webpack packages, and refactored everything needed to support Firebase v9 beta. Obviously these changes will break anything NOT using v9 Beta, so a major update with all those breaking changes wouldn't be a slick move. But, I am more than happy to get something together so we could have a seperate release specifically for v9 that would be available in addition to the current RFH package which is awesome. So far I have all Auth features, storage features, and a couple Firestore features tested and running smooth without any errors or dependency issues. I'm here to help out if this something you all would deem useful! |
Did you remove the compat to be complete modular? From firebase docs:
|
I did ** Also, Firebase did make it clear that we would be moving to 9, and once that happens v8 will become frozen- not sure when, but it'd be sweet to at least have it out on the table so anyone can use the library same as usual 😊 |
Firebase V9 is released! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v9 major version is released!
@@ -52,7 +52,7 @@ | |||
"dependencies": {}, | |||
"devDependencies": { | |||
"@types/react": "^17.0.0", | |||
"firebase": "^8.0.0", | |||
"firebase": "9.0.0-beta.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"firebase": "9.0.0-beta.2", | |
"firebase": "9.0.0", |
@@ -67,7 +67,7 @@ | |||
}, | |||
"peerDependencies": { | |||
"react": ">= 16.8.0", | |||
"firebase": ">= 8.0.0" | |||
"firebase": ">= 9.0.0-beta.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"firebase": ">= 9.0.0-beta.2" | |
"firebase": ">= 9.0.0" |
@chrisbianca is there any plans on merging this PR? |
Can Anyone help me !! |
Seems I will have to remove react-firebase-hooks from my project :( |
Would love to see this get merged! |
You guys might wanna check this one out https://github.com/FirebaseExtended/reactfire |
aaaaaaaaaaaaanything new about merging this PR? |
Please merge this PR 👍 |
👋 All my projects are also stuck on firebase 8, and this PR doesn't do a full migration to firebase 9 but still uses the compatibility imports. Therefore, I tried to do a full migration of the whole package and opened a separate PR converts all components and the documentation. That PR is temporarily deployed as More details in the PR: #174 |
Not all heroes wear capes. |
We should actually get him a cape tho |
Closed in favour of #174 which doesn't use compat imports. Nice job @andipaetzold |
Adds support for Firebase 9.
Closes #105
Firestore and Database hooks are using the compat imports currently, as I'm not familiar with those APIs. In order to get the full tree-shaking improvements of Firebase 9, these compat imports will need to be updated.