[PR #5237] [MERGED] refactor: database row and cell notification #6722

New issue

Closed

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

mirror commented

2026-03-23 23:16:21 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5237
Author: @appflowy
Created: 5/1/2024
Status: ✅ Merged
Merged: 5/2/2024
Merged by: @appflowy

Base: main ← Head: refactor_database_notification

📝 Commits (5)

40ab3b4 refactor: database row and cell notification
25750cc chore: clippy
f36b2d9 chore: fix test
4db0787 chore: fix test
f1cbc38 chore: fix test

📊 Changes

19 files changed (+304 additions, -232 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/plugins/database/application/view/view_listener.dart (+1 -1)
📝 frontend/appflowy_flutter/test/bloc_test/grid_test/util.dart (+2 -2)
📝 frontend/appflowy_tauri/src-tauri/Cargo.lock (+16 -13)
📝 frontend/appflowy_tauri/src-tauri/Cargo.toml (+7 -7)
📝 frontend/appflowy_web/wasm-libs/Cargo.lock (+10 -7)
📝 frontend/appflowy_web/wasm-libs/Cargo.toml (+7 -7)
📝 frontend/appflowy_web_app/src-tauri/Cargo.lock (+10 -7)
📝 frontend/appflowy_web_app/src-tauri/Cargo.toml (+7 -7)
📝 frontend/rust-lib/Cargo.lock (+10 -7)
📝 frontend/rust-lib/Cargo.toml (+7 -7)
📝 frontend/rust-lib/flowy-database2/src/notification.rs (+3 -3)
📝 frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs (+9 -76)
📝 frontend/rust-lib/flowy-database2/src/services/database/database_observe.rs (+55 -10)
📝 frontend/rust-lib/flowy-database2/src/services/database_view/notifier.rs (+1 -1)
📝 frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs (+3 -3)
📝 frontend/rust-lib/flowy-notification/Cargo.toml (+3 -0)
➕ frontend/rust-lib/flowy-notification/src/builder.rs (+94 -0)
➕ frontend/rust-lib/flowy-notification/src/debounce.rs (+54 -0)
📝 frontend/rust-lib/flowy-notification/src/lib.rs (+5 -74)

📄 Description

Using yrs observer as a trigger to send notification to the frontend. Using this approach, it can auto refresh the UI after receiving remote changes

TODO:
migrating all notification

_{🔄 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/5237 **Author:** [@appflowy](https://github.com/appflowy) **Created:** 5/1/2024 **Status:** ✅ Merged **Merged:** 5/2/2024 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `refactor_database_notification` --- ### 📝 Commits (5) - [`40ab3b4`](https://github.com/AppFlowy-IO/AppFlowy/commit/40ab3b45f25fdc6cb3db7d5776747a855a2b80a0) refactor: database row and cell notification - [`25750cc`](https://github.com/AppFlowy-IO/AppFlowy/commit/25750cce0886c8e3a016e782e7d1df2c783445de) chore: clippy - [`f36b2d9`](https://github.com/AppFlowy-IO/AppFlowy/commit/f36b2d9b303cb33c763c5f0ea45f5f7d71e094cf) chore: fix test - [`4db0787`](https://github.com/AppFlowy-IO/AppFlowy/commit/4db07872e97eb3ed42efb8ebb3e0c566251b6d95) chore: fix test - [`f1cbc38`](https://github.com/AppFlowy-IO/AppFlowy/commit/f1cbc3877332035bbd067c556232067836c57714) chore: fix test ### 📊 Changes **19 files changed** (+304 additions, -232 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/plugins/database/application/view/view_listener.dart` (+1 -1) 📝 `frontend/appflowy_flutter/test/bloc_test/grid_test/util.dart` (+2 -2) 📝 `frontend/appflowy_tauri/src-tauri/Cargo.lock` (+16 -13) 📝 `frontend/appflowy_tauri/src-tauri/Cargo.toml` (+7 -7) 📝 `frontend/appflowy_web/wasm-libs/Cargo.lock` (+10 -7) 📝 `frontend/appflowy_web/wasm-libs/Cargo.toml` (+7 -7) 📝 `frontend/appflowy_web_app/src-tauri/Cargo.lock` (+10 -7) 📝 `frontend/appflowy_web_app/src-tauri/Cargo.toml` (+7 -7) 📝 `frontend/rust-lib/Cargo.lock` (+10 -7) 📝 `frontend/rust-lib/Cargo.toml` (+7 -7) 📝 `frontend/rust-lib/flowy-database2/src/notification.rs` (+3 -3) 📝 `frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs` (+9 -76) 📝 `frontend/rust-lib/flowy-database2/src/services/database/database_observe.rs` (+55 -10) 📝 `frontend/rust-lib/flowy-database2/src/services/database_view/notifier.rs` (+1 -1) 📝 `frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs` (+3 -3) 📝 `frontend/rust-lib/flowy-notification/Cargo.toml` (+3 -0) ➕ `frontend/rust-lib/flowy-notification/src/builder.rs` (+94 -0) ➕ `frontend/rust-lib/flowy-notification/src/debounce.rs` (+54 -0) 📝 `frontend/rust-lib/flowy-notification/src/lib.rs` (+5 -74) </details> ### 📄 Description Using yrs observer as a trigger to send notification to the frontend. Using this approach, it can auto refresh the UI after receiving remote changes TODO: migrating all notification --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>