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

feat(firestore): support for aggregate queries including sum() & average() #8115

Merged
merged 28 commits into from
Nov 14, 2024

test: ensure aggregate fields are exposed to end user

cb54669
Select commit
Loading
Failed to load commit list.
Merged

feat(firestore): support for aggregate queries including sum() & average() #8115

test: ensure aggregate fields are exposed to end user
cb54669
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 14, 2024 in 0s

68.63% of diff hit (target 83.97%)

View this Pull Request on Codecov

68.63% of diff hit (target 83.97%)

Annotations

Check warning on line 46 in packages/firestore/lib/FirestoreAggregate.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/FirestoreAggregate.js#L46

Added line #L46 was not covered by tests

Check warning on line 55 in packages/firestore/lib/FirestoreAggregate.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/FirestoreAggregate.js#L55

Added line #L55 was not covered by tests

Check warning on line 78 in packages/firestore/lib/FirestoreAggregate.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/FirestoreAggregate.js#L78

Added line #L78 was not covered by tests

Check warning on line 84 in packages/firestore/lib/FirestoreAggregate.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/FirestoreAggregate.js#L84

Added line #L84 was not covered by tests

Check warning on line 90 in packages/firestore/lib/FirestoreAggregate.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/FirestoreAggregate.js#L89-L90

Added lines #L89 - L90 were not covered by tests

Check warning on line 203 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L203

Added line #L203 was not covered by tests

Check warning on line 212 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L212

Added line #L212 was not covered by tests

Check warning on line 232 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L230-L232

Added lines #L230 - L232 were not covered by tests

Check warning on line 236 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L236

Added line #L236 was not covered by tests

Check warning on line 241 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L241

Added line #L241 was not covered by tests

Check warning on line 267 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L267

Added line #L267 was not covered by tests

Check warning on line 276 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L276

Added line #L276 was not covered by tests

Check warning on line 284 in packages/firestore/lib/modular/index.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/firestore/lib/modular/index.js#L284

Added line #L284 was not covered by tests