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

Add kanbanAggregateOperation field on view and read it in FE #8752

Closed
ijreilly opened this issue Nov 26, 2024 · 0 comments · Fixed by #8833
Closed

Add kanbanAggregateOperation field on view and read it in FE #8752

ijreilly opened this issue Nov 26, 2024 · 0 comments · Fixed by #8833
Assignees

Comments

@ijreilly
Copy link
Collaborator

ijreilly commented Nov 26, 2024

Scope & Context

kanbanAggregateOperation accepts values from enum AGGREGATE_OPERATIONS.
This value is read in the FE and stored in a recoil state (similar to recordIndexKanbanFieldMetadataIdState).

It should also be used when creating a new view (duplicate view or create new view from view with filter).

@ijreilly ijreilly changed the title Add kanbanAggregateOperation field on view table + feature flag Add kanbanAggregateOperation field on view and read it in FE + feature flag Nov 26, 2024
@ijreilly ijreilly self-assigned this Nov 26, 2024
@ijreilly ijreilly changed the title Add kanbanAggregateOperation field on view and read it in FE + feature flag Add kanbanAggregateOperation field on view and read it in FE Nov 26, 2024
@charlesBochet charlesBochet moved this from 🆕 New to 🔖 Planned in Product development ✅ Nov 26, 2024
@ijreilly ijreilly moved this from 🔖 Planned to 🏗 In progress in Product development ✅ Nov 26, 2024
Weiko added a commit that referenced this issue Dec 3, 2024
Closes #8752, #8753, #8754

Implements usage of aggregate queries in kanban views.

https://github.com/user-attachments/assets/732590ca-2785-4c57-82d5-d999a2279e92

TO DO

1. write tests + storybook
2. Fix values displayed should have the same format as defined in number
fields + Fix display for amountMicros

---------

Co-authored-by: Weiko <[email protected]>
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in Product development ✅ Dec 3, 2024
mdrazak2001 pushed a commit to mdrazak2001/twenty that referenced this issue Dec 4, 2024
Closes twentyhq#8752, twentyhq#8753, twentyhq#8754

Implements usage of aggregate queries in kanban views.

https://github.com/user-attachments/assets/732590ca-2785-4c57-82d5-d999a2279e92

TO DO

1. write tests + storybook
2. Fix values displayed should have the same format as defined in number
fields + Fix display for amountMicros

---------

Co-authored-by: Weiko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant