[PR #581] [MERGED] Refactor/grid filter #4212

New issue

Closed

opened 2026-03-23 21:36:48 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 21:36:48 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/581
Author: @appflowy
Created: 7/2/2022
Status: ✅ Merged
Merged: 7/2/2022
Merged by: @appflowy

Base: main ← Head: refactor/grid_filter

📝 Commits (5)

4d8f101 chore: gen filter task
50d0eff chore: cache filter result
a9a797d refactor: wrap FieldRevision with Arc
c1ea97c refactor: move entities from flowy-grid-data-model to flowy-grid crate
9803fe1 refactor: grid block notification

📊 Changes

107 files changed (+2201 additions, -1640 deletions)

View changed files

📝 frontend/app_flowy/lib/workspace/application/doc/doc_service.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/block/block_listener.dart (+1 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/cell/cell_service/cell_service.dart (+1 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/cell/date_cell_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/field_action_sheet_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/field_cell_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/field_editor_pannel_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/field_listener.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/field_service.dart (+1 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/field/grid_listenr.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/type_option/number_format_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/field/type_option/type_option_service.dart (+0 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/grid_bloc.dart (+4 -3)
📝 frontend/app_flowy/lib/workspace/application/grid/grid_header_bloc.dart (+1 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/grid_service.dart (+4 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/row/row_action_sheet_bloc.dart (+5 -1)
📝 frontend/app_flowy/lib/workspace/application/grid/row/row_bloc.dart (+6 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/row/row_listener.dart (+2 -2)
📝 frontend/app_flowy/lib/workspace/application/grid/row/row_service.dart (+52 -39)
📝 frontend/app_flowy/lib/workspace/application/grid/setting/property_bloc.dart (+1 -1)

...and 80 more files

📄 Description

Cache filter result
Wrap FieldRevision with Arc to reduce the memory cost
Move flowy-grid-data-model's entities to the flowy-grid crate
Config grid block notification

The grid filter is not finished yet, just merged to reduce PR size.

_{🔄 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/581 **Author:** [@appflowy](https://github.com/appflowy) **Created:** 7/2/2022 **Status:** ✅ Merged **Merged:** 7/2/2022 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `refactor/grid_filter` --- ### 📝 Commits (5) - [`4d8f101`](https://github.com/AppFlowy-IO/AppFlowy/commit/4d8f101a42577222ebe87baf09ecc8b37e8bf6c7) chore: gen filter task - [`50d0eff`](https://github.com/AppFlowy-IO/AppFlowy/commit/50d0eff0392fb07c9c37215afc87ac123654bc87) chore: cache filter result - [`a9a797d`](https://github.com/AppFlowy-IO/AppFlowy/commit/a9a797ddaf853fac26da03124f2d3193cd21b89f) refactor: wrap FieldRevision with Arc - [`c1ea97c`](https://github.com/AppFlowy-IO/AppFlowy/commit/c1ea97c72900e8dc8f117b43ce96e9d238d99f31) refactor: move entities from flowy-grid-data-model to flowy-grid crate - [`9803fe1`](https://github.com/AppFlowy-IO/AppFlowy/commit/9803fe1ccbc12458cc901918ccbad944c6748009) refactor: grid block notification ### 📊 Changes **107 files changed** (+2201 additions, -1640 deletions) <details> <summary>View changed files</summary> 📝 `frontend/app_flowy/lib/workspace/application/doc/doc_service.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/block/block_listener.dart` (+1 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/cell/cell_service/cell_service.dart` (+1 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/cell/date_cell_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/field_action_sheet_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/field_cell_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/field_editor_pannel_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/field_listener.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/field_service.dart` (+1 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/grid_listenr.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/type_option/number_format_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/field/type_option/type_option_service.dart` (+0 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/grid_bloc.dart` (+4 -3) 📝 `frontend/app_flowy/lib/workspace/application/grid/grid_header_bloc.dart` (+1 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/grid_service.dart` (+4 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/row/row_action_sheet_bloc.dart` (+5 -1) 📝 `frontend/app_flowy/lib/workspace/application/grid/row/row_bloc.dart` (+6 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/row/row_listener.dart` (+2 -2) 📝 `frontend/app_flowy/lib/workspace/application/grid/row/row_service.dart` (+52 -39) 📝 `frontend/app_flowy/lib/workspace/application/grid/setting/property_bloc.dart` (+1 -1) _...and 80 more files_ </details> ### 📄 Description 1. Cache filter result 2. Wrap FieldRevision with Arc to reduce the memory cost 3. Move flowy-grid-data-model's entities to the flowy-grid crate 4. Config grid block notification The grid filter is not finished yet, just merged to reduce PR size. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>