-
Notifications
You must be signed in to change notification settings - Fork 937
Add long-alias support for aggregations. #7244
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
Add long-alias support for aggregations. #7244
Conversation
|
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1This report is too large (638,604 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
Not sure why CI is failing with |
@ehsannas, I made all of these changes. It took me a bit, but now I agree that this is a better approach. Thanks for your review. |
…kduckworth/sum-avg-4
Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:
Discussion
If not, go file an issue about this before creating a pull request to discuss.
Testing
API Changes
us make Firebase APIs better, please propose your change in an issue so that we
can discuss it together.