-
Notifications
You must be signed in to change notification settings - Fork 26
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
Feature request - Firestore #25
Comments
Sorry, only just saw this. The support for Firestore is difficult to integrate with R given how it works. I tried, it's not impossible but makes for a very ugly API |
It would be awesome if firestore feature is available even with an ugly API. Thanks for your package. |
Have you thought about integrating something like this: |
Yes, I have used the functions from that post and also wrote my own for query filtering. It would be awesome if {firebase} also has these available. |
The issue is not technical but how we manage to nicely wrap the query API. We cannot do HTTP requests, {firebase} is all based on the official JavaScript SDK's and I think it should remain like this. If you anyone has anything to propose in that regard it is more than welcome, e.g.: take a look at this, I think it shows some of the complexities I hint at. We need an R API not just around R/shiny (sendCustomMessage) we need to map the firestore ORM in R, and that's a lot of work I'm not sure where to start. |
I understand. I'd love to contribute but I have little to no experience with JavaScript. I guess the best way to go is to use the Cloud Firestore REST API using {httr} as I am doing now. Thank you for your thorough response. |
Initial work on firestore support on a new branch remotes::install_github("JohnCoene/firebase@firestore") See this document if you want to try |
This is really cool, thanks @JohnCoene. I understand that {firebase} won't use HTTP requests. So, if anyone else is interested, I authored this package based on Firestore REST API: https://github.com/Presage-Group/frstore But {firebase} native solution for Firestore will be awesome. |
Hello!
There is any plan to add support for Firestore database ?
The text was updated successfully, but these errors were encountered: