search

funwithflutter / lit_firebase_auth

Lit Firebase Auth simplifies the process of adding Firebase Authentication to a Flutter application.
MIT License
76 stars 18 forks source link

Compatibility with cloud_firestore@0.14.0 #8

Closed ciriousjoker closed 4 years ago

ciriousjoker commented 4 years ago

Not sure if it's the right time to upgrade, but 0.14.0 exists now, so I guess at some point it will be.

funwithflutter commented 4 years ago

Yes, definitely. This is a top priority to get the package to use the latest FlutterFire packages.

ciriousjoker commented 4 years ago

Since I suddenly needed this, I created a fork and made the changes. Do you want me to create a PR for this?

It's slightly breaking, since I had to rename User to LitUser because freezed doesn't support prefixes. https://github.com/rrousselGit/freezed/issues/70

funwithflutter commented 4 years ago

Since I suddenly needed this, I created a fork and made the changes. Do you want me to create a PR for this?

It's slightly breaking, since I had to rename User to LitUser because freezed doesn't support prefixes. rrousselGit/freezed#70

Yes, definitely make a pull request! In hindsight it might be better to just return the FirebaseAuth User, instead of mapping to a new User. But for now, you can rename it to LitUser. Thanks 😀