-
-
Notifications
You must be signed in to change notification settings - Fork 175
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
Be able to override FirestoreClient::DEFAULT_DATABASE #205
Comments
At the moment, the Firestore (and Cloud Storage) component are just super simple bridges, allowing them to be configured with options, as setting the default database) would be a good addition, thank you for bringing it up! Todo list for myself:
I only have limited time available, so I can't give you an ETA, but it's on my radar. |
Oh thank you very much @jeromegamez! Awesome! |
I didn't do it in the Laravel package yet 😅🙈 |
I just realized it 😫 |
Happy New Year @jeromegamez! I would like to know if any update will be done about this issue in the current month? Thank you very much! |
Since my work on my Firebase projects isn't funded (you've probably seen it in the readme of the SDK repo), I'm not motivated to do more than keeping everything stable and working at the moment, so I can't give you an ETA. I understand you need it for your business and to make money, so for the moment I'd recommend forking the package and changing what you need - perhaps you'll do it in a way that can be merged back into here. |
I understand. I will fork it so. |
Describe the feature you would like to see
As described in https://cloud.google.com/firestore/docs/manage-databases?hl=fr#create_a_database, I would like to be able to override the constant defined in
FirestoreClient
namedDEFAULT_DATABASE
.The goal will be to allow defining it via the configuration file.
My application uses multi-tenancy, and I want to switch the configuration while bootstrapping or reverting the tenancy.
I thought I just had to extend the Factory class (impossible because it's final), or whatever, but I couldn't accomplish this.
Maybe is it another way to do this?
Thank you for your awesome work in this package.
Used versions:
The text was updated successfully, but these errors were encountered: