[PR #3123] [MERGED] refactor: remove shared instance KV #5580

New issue

Closed

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

mirror commented

2026-03-23 22:19:22 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3123
Author: @appflowy
Created: 8/6/2023
Status: ✅ Merged
Merged: 8/6/2023
Merged by: @appflowy

Base: main ← Head: refactor/remove_shared_instance_kv

📝 Commits (2)

f7ac493 refactor: remove shared instance KV
c5b0d13 test: enable document test

📊 Changes

13 files changed (+246 additions, -153 deletions)

View changed files

📝 frontend/rust-lib/flowy-config/src/event_handler.rs (+41 -18)
📝 frontend/rust-lib/flowy-config/src/event_map.rs (+5 -1)
📝 frontend/rust-lib/flowy-core/src/integrate/server.rs (+22 -10)
📝 frontend/rust-lib/flowy-core/src/lib.rs (+18 -13)
📝 frontend/rust-lib/flowy-core/src/module.rs (+5 -1)
📝 frontend/rust-lib/flowy-sqlite/src/kv/kv.rs (+68 -72)
📝 frontend/rust-lib/flowy-test/src/lib.rs (+11 -11)
📝 frontend/rust-lib/flowy-test/tests/user/migration_test/document_test.rs (+3 -5)
📝 frontend/rust-lib/flowy-test/tests/user/migration_test/mod.rs (+1 -1)
📝 frontend/rust-lib/flowy-user/src/event_handler.rs (+23 -7)
📝 frontend/rust-lib/flowy-user/src/event_map.rs (+5 -0)
📝 frontend/rust-lib/flowy-user/src/migrations/historical_document.rs (+7 -4)
📝 frontend/rust-lib/flowy-user/src/services/user_session.rs (+37 -10)

📄 Description

PR Checklist

My code adheres to the AppFlowy Style Guide
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/3123 **Author:** [@appflowy](https://github.com/appflowy) **Created:** 8/6/2023 **Status:** ✅ Merged **Merged:** 8/6/2023 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `refactor/remove_shared_instance_kv` --- ### 📝 Commits (2) - [`f7ac493`](https://github.com/AppFlowy-IO/AppFlowy/commit/f7ac493cedc11ca724e3fa4af4ea07156befb89c) refactor: remove shared instance KV - [`c5b0d13`](https://github.com/AppFlowy-IO/AppFlowy/commit/c5b0d13d7a6df1a24bc87bd583e5a803bc9197de) test: enable document test ### 📊 Changes **13 files changed** (+246 additions, -153 deletions) <details> <summary>View changed files</summary> 📝 `frontend/rust-lib/flowy-config/src/event_handler.rs` (+41 -18) 📝 `frontend/rust-lib/flowy-config/src/event_map.rs` (+5 -1) 📝 `frontend/rust-lib/flowy-core/src/integrate/server.rs` (+22 -10) 📝 `frontend/rust-lib/flowy-core/src/lib.rs` (+18 -13) 📝 `frontend/rust-lib/flowy-core/src/module.rs` (+5 -1) 📝 `frontend/rust-lib/flowy-sqlite/src/kv/kv.rs` (+68 -72) 📝 `frontend/rust-lib/flowy-test/src/lib.rs` (+11 -11) 📝 `frontend/rust-lib/flowy-test/tests/user/migration_test/document_test.rs` (+3 -5) 📝 `frontend/rust-lib/flowy-test/tests/user/migration_test/mod.rs` (+1 -1) 📝 `frontend/rust-lib/flowy-user/src/event_handler.rs` (+23 -7) 📝 `frontend/rust-lib/flowy-user/src/event_map.rs` (+5 -0) 📝 `frontend/rust-lib/flowy-user/src/migrations/historical_document.rs` (+7 -4) 📝 `frontend/rust-lib/flowy-user/src/services/user_session.rs` (+37 -10) </details> ### 📄 Description #### PR Checklist - [ ] My code adheres to the [AppFlowy Style Guide](https://appflowy.gitbook.io/docs/essential-documentation/contribute-to-appflowy/software-contributions/submitting-code/style-guides) - [ ] 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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>