-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Aggregate queries #6868
Labels
Comments
FelixMalfait
added
the
scope: backend
Issues that are affecting the backend side only
label
Sep 3, 2024
CleanShot.2024-09-19.at.13.57.53.mp4 |
Merged
Weiko
added a commit
that referenced
this issue
Nov 14, 2024
First step of #6868 Adds min.., max.. queries for DATETIME fields adds min.., max.., avg.., sum.. queries for NUMBER fields (count distinct operation and composite fields such as CURRENCY handling will be dealt with in a future PR) <img width="1422" alt="Capture d’écran 2024-11-06 à 15 48 46" src="https://github.com/user-attachments/assets/4bcdece0-ad3e-4536-9720-fe4044a36719"> --------- Co-authored-by: Charles Bochet <[email protected]> Co-authored-by: Weiko <[email protected]>
khuddite
pushed a commit
to khuddite/twenty
that referenced
this issue
Nov 18, 2024
First step of twentyhq#6868 Adds min.., max.. queries for DATETIME fields adds min.., max.., avg.., sum.. queries for NUMBER fields (count distinct operation and composite fields such as CURRENCY handling will be dealt with in a future PR) <img width="1422" alt="Capture d’écran 2024-11-06 à 15 48 46" src="https://github.com/user-attachments/assets/4bcdece0-ad3e-4536-9720-fe4044a36719"> --------- Co-authored-by: Charles Bochet <[email protected]> Co-authored-by: Weiko <[email protected]>
@Bonapara do we have further designs ? |
Do you still need anything, @ijreilly? |
no all good for now after our talk ! @Bonapara |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Our GraphQL API should support aggregate queries on findMany queries.
We already have
totalCount
so we can take inspiration from that.But it's significantly more challenging because the aggregated fields name are dynamic and depend on the object names themselves. Here's how it could look like:
We'll need to add:
min
,max
,avg
,sum
,countDistinct
The text was updated successfully, but these errors were encountered: