[PR #4688] [MERGED] feat: advanced filters backend logic #6388

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

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4688
Author: @richardshiue
Created: 2/21/2024
Status: Merged
Merged: 3/14/2024
Merged by: @appflowy

Base: mainHead: feat/advanced-filters


📝 Commits (9)

  • c402161 feat: implement advanced filters
  • d46c037 test: adapt tests to changes
  • 626f6df test: add advanced filter tests
  • 9b32bc4 chore: adapt flutter frontend to changes
  • e9bc143 chore: adapt tauri frontend to changes
  • 04934d7 chore: bump collab
  • 4cf272f chore: launch review
  • de4d178 chore: merge remote-tracking branch 'upstream/main' into feat/advanced-filters
  • 0d8d4a8 chore: merge remote-tracking branch 'upstream/main' into feat/advanced-filters

📊 Changes

50 files changed (+1881 additions, -1480 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/plugins/database/application/field/field_controller.dart (+15 -80)
📝 frontend/appflowy_flutter/lib/plugins/database/domain/filter_listener.dart (+5 -24)
📝 frontend/appflowy_flutter/lib/plugins/database/domain/filter_service.dart (+138 -75)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/checkbox_filter_editor_bloc.dart (+3 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/checklist_filter_bloc.dart (+0 -4)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/number_filter_editor_bloc.dart (+0 -6)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/select_option_filter_bloc.dart (+0 -4)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/text_filter_editor_bloc.dart (+3 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/filter/filter_info.dart (+14 -14)
📝 frontend/appflowy_flutter/test/bloc_test/grid_test/filter/create_filter_test.dart (+0 -1)
📝 frontend/appflowy_flutter/test/bloc_test/grid_test/filter/filter_rows_by_text_test.dart (+0 -4)
📝 frontend/appflowy_tauri/src-tauri/Cargo.toml (+2 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_listeners.ts (+3 -30)
📝 frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_service.ts (+12 -15)
📝 frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_types.ts (+3 -3)
📝 frontend/rust-lib/flowy-database2/src/entities/filter_entities/checkbox_filter.rs (+6 -19)
📝 frontend/rust-lib/flowy-database2/src/entities/filter_entities/checklist_filter.rs (+5 -17)
📝 frontend/rust-lib/flowy-database2/src/entities/filter_entities/date_filter.rs (+7 -27)
📝 frontend/rust-lib/flowy-database2/src/entities/filter_entities/filter_changeset.rs (+6 -38)
📝 frontend/rust-lib/flowy-database2/src/entities/filter_entities/number_filter.rs (+5 -17)

...and 30 more files

📄 Description

This PR contains only the backend logic. a tech design write up is in the gantt.

Requires https://github.com/AppFlowy-IO/AppFlowy-Collab/pull/159

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/4688 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 2/21/2024 **Status:** ✅ Merged **Merged:** 3/14/2024 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `feat/advanced-filters` --- ### 📝 Commits (9) - [`c402161`](https://github.com/AppFlowy-IO/AppFlowy/commit/c402161cce980f8225558d23b4bc94b73e7a7d97) feat: implement advanced filters - [`d46c037`](https://github.com/AppFlowy-IO/AppFlowy/commit/d46c0377cace45680f461181738f510849827d16) test: adapt tests to changes - [`626f6df`](https://github.com/AppFlowy-IO/AppFlowy/commit/626f6df128bbfb6e69696782ac92113bef06227b) test: add advanced filter tests - [`9b32bc4`](https://github.com/AppFlowy-IO/AppFlowy/commit/9b32bc44c6d942f47d409b8c6c7d93dfb6f4d6d8) chore: adapt flutter frontend to changes - [`e9bc143`](https://github.com/AppFlowy-IO/AppFlowy/commit/e9bc1437d6bbd3568124f61fbf0ae9b8f7ebdefc) chore: adapt tauri frontend to changes - [`04934d7`](https://github.com/AppFlowy-IO/AppFlowy/commit/04934d789ff538fb592cfbca7e9c3b35033cf87d) chore: bump collab - [`4cf272f`](https://github.com/AppFlowy-IO/AppFlowy/commit/4cf272f31b0e433bff731d4df5049b6aada11da7) chore: launch review - [`de4d178`](https://github.com/AppFlowy-IO/AppFlowy/commit/de4d178c0c5be8cc5d03f7aa57e96636135d651f) chore: merge remote-tracking branch 'upstream/main' into feat/advanced-filters - [`0d8d4a8`](https://github.com/AppFlowy-IO/AppFlowy/commit/0d8d4a8294e8f7552932fe919a6ca66fb8c11187) chore: merge remote-tracking branch 'upstream/main' into feat/advanced-filters ### 📊 Changes **50 files changed** (+1881 additions, -1480 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/plugins/database/application/field/field_controller.dart` (+15 -80) 📝 `frontend/appflowy_flutter/lib/plugins/database/domain/filter_listener.dart` (+5 -24) 📝 `frontend/appflowy_flutter/lib/plugins/database/domain/filter_service.dart` (+138 -75) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/checkbox_filter_editor_bloc.dart` (+3 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/checklist_filter_bloc.dart` (+0 -4) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/number_filter_editor_bloc.dart` (+0 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/select_option_filter_bloc.dart` (+0 -4) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/filter/text_filter_editor_bloc.dart` (+3 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/filter/filter_info.dart` (+14 -14) 📝 `frontend/appflowy_flutter/test/bloc_test/grid_test/filter/create_filter_test.dart` (+0 -1) 📝 `frontend/appflowy_flutter/test/bloc_test/grid_test/filter/filter_rows_by_text_test.dart` (+0 -4) 📝 `frontend/appflowy_tauri/src-tauri/Cargo.toml` (+2 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_listeners.ts` (+3 -30) 📝 `frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_service.ts` (+12 -15) 📝 `frontend/appflowy_tauri/src/appflowy_app/application/database/filter/filter_types.ts` (+3 -3) 📝 `frontend/rust-lib/flowy-database2/src/entities/filter_entities/checkbox_filter.rs` (+6 -19) 📝 `frontend/rust-lib/flowy-database2/src/entities/filter_entities/checklist_filter.rs` (+5 -17) 📝 `frontend/rust-lib/flowy-database2/src/entities/filter_entities/date_filter.rs` (+7 -27) 📝 `frontend/rust-lib/flowy-database2/src/entities/filter_entities/filter_changeset.rs` (+6 -38) 📝 `frontend/rust-lib/flowy-database2/src/entities/filter_entities/number_filter.rs` (+5 -17) _...and 30 more files_ </details> ### 📄 Description This PR contains only the backend logic. a tech design write up is in the gantt. Requires https://github.com/AppFlowy-IO/AppFlowy-Collab/pull/159 ### 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. - [ ] 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:23:00 +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#6388
No description provided.