[PR #4473] [MERGED] feat: calculations #6263

Closed
opened 2026-03-23 22:22:26 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4473
Author: @Xazin
Created: 1/23/2024
Status: Merged
Merged: 2/3/2024
Merged by: @appflowy

Base: mainHead: feat/calculation-controller


📝 Commits (10+)

  • 6c3aa8a feat: initial calculation controller
  • 49ef833 chore: merge branch 'upstream/main' into feat/calculation-controller
  • 97b2a10 fix: entities
  • afbd31e chore: merge branch 'upstream/main' into feat/calculation-controller
  • 89f5eaa feat: calculations
  • 366dce8 chore: merge branch 'upstream/main' into feat/calculation-controller
  • a8d8d3f fix: review comments and support floats
  • 1c7be27 fix: abstract business logic into calculations service
  • 287775d chore: merge branch 'upstream/main' into feat/calculation-controller
  • 64cb71e fix: clean calculation entities after merge

📊 Changes

65 files changed (+2456 additions, -220 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/database/database_field_test.dart (+1 -6)
frontend/appflowy_flutter/integration_test/grid/grid_calculations_test.dart (+107 -0)
📝 frontend/appflowy_flutter/integration_test/util/database_test_op.dart (+28 -0)
frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculation_type_ext.dart (+18 -0)
frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculations_listener.dart (+54 -0)
frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculations_service.dart (+46 -0)
frontend/appflowy_flutter/lib/plugins/database/grid/application/calculations/calculations_bloc.dart (+182 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/grid_page.dart (+22 -16)
frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculate_cell.dart (+137 -0)
frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculation_selector.dart (+61 -0)
frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculation_type_item.dart (+33 -0)
frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculations_row.dart (+46 -0)
frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/remove_calculation_button.dart (+34 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/footer/grid_footer.dart (+9 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/row.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_detail.dart (+4 -4)
📝 frontend/appflowy_tauri/src-tauri/Cargo.lock (+8 -7)
📝 frontend/appflowy_tauri/src-tauri/Cargo.toml (+7 -7)
📝 frontend/appflowy_web/wasm-libs/Cargo.toml (+7 -7)
📝 frontend/resources/translations/en.json (+11 -2)

...and 45 more files

📄 Description

Relates: https://github.com/AppFlowy-IO/AppFlowy-Collab/pull/152

Feature Preview


PR Checklist

  • My code adheres to AppFlowy's Conventions
  • I've listed at least one issue that this PR fixes in the description above.
  • I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • All existing tests are passing.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/4473 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 1/23/2024 **Status:** ✅ Merged **Merged:** 2/3/2024 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `feat/calculation-controller` --- ### 📝 Commits (10+) - [`6c3aa8a`](https://github.com/AppFlowy-IO/AppFlowy/commit/6c3aa8a7e227a7b74bfd3a7ff56738e7d9c8bf27) feat: initial calculation controller - [`49ef833`](https://github.com/AppFlowy-IO/AppFlowy/commit/49ef833a5d350d00e859746b0ba813824b581596) chore: merge branch 'upstream/main' into feat/calculation-controller - [`97b2a10`](https://github.com/AppFlowy-IO/AppFlowy/commit/97b2a100de0ffc7fae86996d543241f4ac50ec0c) fix: entities - [`afbd31e`](https://github.com/AppFlowy-IO/AppFlowy/commit/afbd31e624b97990b3eac9706dede917d34533f0) chore: merge branch 'upstream/main' into feat/calculation-controller - [`89f5eaa`](https://github.com/AppFlowy-IO/AppFlowy/commit/89f5eaa8fa50af283bcb91abc11e2a8d64334f19) feat: calculations - [`366dce8`](https://github.com/AppFlowy-IO/AppFlowy/commit/366dce898656d958de2e7fd90606d8eab7a5bfaf) chore: merge branch 'upstream/main' into feat/calculation-controller - [`a8d8d3f`](https://github.com/AppFlowy-IO/AppFlowy/commit/a8d8d3feaeb93b2f0fb84e817ce48f76c4ee768d) fix: review comments and support floats - [`1c7be27`](https://github.com/AppFlowy-IO/AppFlowy/commit/1c7be2752f6d3729edb702d93838dbe682487c40) fix: abstract business logic into calculations service - [`287775d`](https://github.com/AppFlowy-IO/AppFlowy/commit/287775da9b466862c8166ac9ea7d00120432d422) chore: merge branch 'upstream/main' into feat/calculation-controller - [`64cb71e`](https://github.com/AppFlowy-IO/AppFlowy/commit/64cb71e3f4c4e13e90a03719da8743249485fd20) fix: clean calculation entities after merge ### 📊 Changes **65 files changed** (+2456 additions, -220 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/database/database_field_test.dart` (+1 -6) ➕ `frontend/appflowy_flutter/integration_test/grid/grid_calculations_test.dart` (+107 -0) 📝 `frontend/appflowy_flutter/integration_test/util/database_test_op.dart` (+28 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculation_type_ext.dart` (+18 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculations_listener.dart` (+54 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/application/calculations/calculations_service.dart` (+46 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/application/calculations/calculations_bloc.dart` (+182 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/grid_page.dart` (+22 -16) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculate_cell.dart` (+137 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculation_selector.dart` (+61 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculation_type_item.dart` (+33 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculations_row.dart` (+46 -0) ➕ `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/remove_calculation_button.dart` (+34 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/footer/grid_footer.dart` (+9 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/row.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_detail.dart` (+4 -4) 📝 `frontend/appflowy_tauri/src-tauri/Cargo.lock` (+8 -7) 📝 `frontend/appflowy_tauri/src-tauri/Cargo.toml` (+7 -7) 📝 `frontend/appflowy_web/wasm-libs/Cargo.toml` (+7 -7) 📝 `frontend/resources/translations/en.json` (+11 -2) _...and 45 more files_ </details> ### 📄 Description Relates: https://github.com/AppFlowy-IO/AppFlowy-Collab/pull/152 ### Feature Preview <!--- List at least one issue here that this PR addresses. If it fixes the issue, please use the [fixes](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests) keyword to close the issue. For example: fixes https://github.com/AppFlowy-IO/AppFlowy/pull/2106 --> --- <!--- Before you mark this PR ready for review, run through this checklist! --> #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror 2026-03-23 22:22:26 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#6263
No description provided.