[PR #1273] [MERGED] feat: notify transaction instead of operations #4639

New issue

Closed

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

mirror commented

2026-03-23 21:38:41 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/1273
Author: @LucasXu0
Created: 10/13/2022
Status: ✅ Merged
Merged: 10/14/2022
Merged by: @LucasXu0

Base: main ← Head: transform

📝 Commits (3)

8895430 chore: update locale
71b1769 feat: notify transaction instead of operations
f3eeb47 refactor: abstract the common transaction to commands

📊 Changes

40 files changed (+563 additions, -549 deletions)

View changed files

📝 frontend/app_flowy/packages/appflowy_editor/example/lib/main.dart (+2 -2)
📝 frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/code_block_node_widget.dart (+12 -11)
📝 frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/horizontal_rule_node_widget.dart (+6 -6)
📝 frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/tex_block_node_widget.dart (+13 -11)
📝 frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/underscore_to_italic.dart (+2 -2)
➕ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/attributes_commands.dart (+20 -0)
➕ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/command_extension.dart (+54 -0)
➖ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/edit_text.dart (+0 -33)
➖ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/format_built_in_text.dart (+0 -83)
➖ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/format_text.dart (+0 -64)
➕ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/text/text_commands.dart (+98 -0)
➖ frontend/app_flowy/packages/appflowy_editor/lib/src/commands/text_command_infra.dart (+0 -43)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/core/transform/transaction.dart (+4 -4)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/editor_state.dart (+9 -20)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/history/undo_manager.dart (+0 -0)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_all.dart (+4 -0)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_de-DE.dart (+19 -16)
➕ frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_hi-IN.dart (+43 -0)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_ru-RU.dart (+18 -16)
📝 frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/l10n.dart (+1 -0)

...and 20 more files

📄 Description

No description provided

_{🔄 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/1273 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 10/13/2022 **Status:** ✅ Merged **Merged:** 10/14/2022 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `transform` --- ### 📝 Commits (3) - [`8895430`](https://github.com/AppFlowy-IO/AppFlowy/commit/8895430d9e24af6be1da0d7cee71afa2d4183246) chore: update locale - [`71b1769`](https://github.com/AppFlowy-IO/AppFlowy/commit/71b1769eee6085fc48167e6e14961bf1fabbf8b1) feat: notify transaction instead of operations - [`f3eeb47`](https://github.com/AppFlowy-IO/AppFlowy/commit/f3eeb471e77028ab0ae16fcc6bf6f162d289b7f0) refactor: abstract the common transaction to commands ### 📊 Changes **40 files changed** (+563 additions, -549 deletions) <details> <summary>View changed files</summary> 📝 `frontend/app_flowy/packages/appflowy_editor/example/lib/main.dart` (+2 -2) 📝 `frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/code_block_node_widget.dart` (+12 -11) 📝 `frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/horizontal_rule_node_widget.dart` (+6 -6) 📝 `frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/tex_block_node_widget.dart` (+13 -11) 📝 `frontend/app_flowy/packages/appflowy_editor/example/lib/plugin/underscore_to_italic.dart` (+2 -2) ➕ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/attributes_commands.dart` (+20 -0) ➕ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/command_extension.dart` (+54 -0) ➖ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/edit_text.dart` (+0 -33) ➖ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/format_built_in_text.dart` (+0 -83) ➖ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/format_text.dart` (+0 -64) ➕ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/text/text_commands.dart` (+98 -0) ➖ `frontend/app_flowy/packages/appflowy_editor/lib/src/commands/text_command_infra.dart` (+0 -43) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/core/transform/transaction.dart` (+4 -4) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/editor_state.dart` (+9 -20) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/history/undo_manager.dart` (+0 -0) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_all.dart` (+4 -0) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_de-DE.dart` (+19 -16) ➕ `frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_hi-IN.dart` (+43 -0) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/intl/messages_ru-RU.dart` (+18 -16) 📝 `frontend/app_flowy/packages/appflowy_editor/lib/src/l10n/l10n.dart` (+1 -0) _...and 20 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>