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

Firebase queue for Firestore? #7

Closed
EECOLOR opened this issue Sep 6, 2018 · 4 comments
Closed

Firebase queue for Firestore? #7

EECOLOR opened this issue Sep 6, 2018 · 4 comments
Labels

Comments

@EECOLOR
Copy link
Member

EECOLOR commented Sep 6, 2018

#5 (comment)

@LoveMHz:

@EECOLOR What at do you purpose is the best solution? A new project or try to expand firebase-queue to support Firestore?

I've managed to hack together Firestore support, but not actually sure it fits in with the current project.

@EECOLOR
Copy link
Member Author

EECOLOR commented Sep 6, 2018

@LoveMHz

Great question!

I do not think this library can support Firestore. There are a few reasons:

  • The Firestore API is very different from the Realtime database API
  • There is (at the moment of writing) no way to locally test Firestore

Another thing that might be problematic is that we are not using Firestore in any of our projects. This would have a negative effect if we had Firestore support in this project:

  • We would need to completely rely on outside contributors to maintain that part of the library
  • If someone asked a question we might not be able to answer it

So I think the best thing would be a separate project with maintainers that know how Firestore works. I would suggest to use the same surface API and behavior (as far as possible) as this library. This would benefit end-users (developers) and allows them to switch their application to another store. If it helps feel free to fork this library and convert it into a Firestore version!

@EECOLOR
Copy link
Member Author

EECOLOR commented Sep 6, 2018

Ohw, I forgot to add: please let me know if you disagree, I try to be open for discussion.

@ElectroBuddha
Copy link

ElectroBuddha commented Sep 14, 2018

It is also possible to use both Realtime DB and Firestore DB in the same project. So the one solution might be to keep parts of your app separated, logic for queuing could use RealtimeDB and the rest of your app could use Firestore DB.

@EECOLOR
Copy link
Member Author

EECOLOR commented Oct 10, 2018

I'm closing this issue. If anyone feels it needs to be reopened, please feel free and don't forget to add a motivation and counter arguments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants