urban-health-labs/CombineFirebase

Update to latest version of Firebase

Opened this issue · 7 comments

ts95 commented

Firebase has received quite a few updates in the past couple of months. Could you update the Firebase dependency so that it points to the latest version (7.3.1 as of this writing)?

kshivang commented

My apologies for responding late, was very busy for couple of weeks. Will look into this by Sunday. Meanwhile you can checkout theses PRs 1st & 2nd regarding this only. Any new idea/suggestion or new PR regarding this would be very much appreciated.

sedestrian commented

Could we have support for firebase > 8.0 please?

staticdreams commented

Any updates on this or the whole idea to support 8.0 is on indefinite standby?

kshivang commented

Should be resolved by now (if you are using with SPM), just invalidate SPM cache.

  • 👍1
sprzenus commented

It isn't released yet. Using master branch you can use firebase 8+ with SPM

rrozenv commented

Hey @sprzenus @kshivang is there any update on this? When adding CombineFirebase as a package dependency (using either branch: master or Up to next major: 0.3.0 on Xcode 13.1 it still uses Firebase 7.11.1.

It does seem like the 8.0.0 release is officially supported without needing to use the master branch: https://github.com/firebase/firebase-ios-sdk/blob/master/SwiftPackageManager.md

Thanks!

sprzenus commented

@rrozenv It worked with the last released version with CocoaPods, but it didn't with SwiftPM.

Note that you have to have both Firebase and CombineFirebase included in the same package manager.

Before merging #20, the framework linked in Swift Package Manager required Firebase in version less than 8. The commit extended supported Firebase version up to 9.0.0 (not included)

However, Firebase in version 8.10.0 added some Combine support. Didn't check that out, though...
You'll find more info in Firebase Release Notes