[PR #4798] [MERGED] fix: update last modified on icon change #6450

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4798
Author: @Xazin
Created: 3/3/2024
Status: ✅ Merged
Merged: 4/9/2024
Merged by: @Xazin

Base: main ← Head: fix/update-icon-last-modified

📝 Commits (7)

3573905 fix: update last modified on icon change
98ed9f7 fix: clippy
b3b5e36 chore: clean code
1f29682 chore: reorder operations
2d9000a chore: merge branch 'upstream/main' into fix/update-icon-last-modified
33af806 chore: merge branch 'upstream/main' into fix/update-icon-last-modified
7240b9e chore: fix import after merge

📊 Changes

4 files changed (+38 additions, -7 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart (+2 -1)
📝 frontend/rust-lib/flowy-database2/src/event_handler.rs (+3 -1)
📝 frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs (+27 -1)
📝 frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs (+6 -4)

📄 Description

Closes: #3409

Feature Preview

https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/32bc5066-8df9-4ef8-87ff-f50dcfe92c46

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/4798 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 3/3/2024 **Status:** ✅ Merged **Merged:** 4/9/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `fix/update-icon-last-modified` --- ### 📝 Commits (7) - [`3573905`](https://github.com/AppFlowy-IO/AppFlowy/commit/35739059ea6eb22d0b36edd18b308fa2bc48db06) fix: update last modified on icon change - [`98ed9f7`](https://github.com/AppFlowy-IO/AppFlowy/commit/98ed9f74e8177dfca066c9cc2c03118857f68c27) fix: clippy - [`b3b5e36`](https://github.com/AppFlowy-IO/AppFlowy/commit/b3b5e36f09955abfc5fe71a6b60a986dc2a58995) chore: clean code - [`1f29682`](https://github.com/AppFlowy-IO/AppFlowy/commit/1f29682854d4a94f47299a398a733d2d05c14474) chore: reorder operations - [`2d9000a`](https://github.com/AppFlowy-IO/AppFlowy/commit/2d9000a2972e6bc6c2a8d3f75cf27a9d0a590c67) chore: merge branch 'upstream/main' into fix/update-icon-last-modified - [`33af806`](https://github.com/AppFlowy-IO/AppFlowy/commit/33af806ade600fc5e35a36092f79e328e2a1d482) chore: merge branch 'upstream/main' into fix/update-icon-last-modified - [`7240b9e`](https://github.com/AppFlowy-IO/AppFlowy/commit/7240b9e01d7ed09e4810196cf95bc2f1684a3eed) chore: fix import after merge ### 📊 Changes **4 files changed** (+38 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart` (+2 -1) 📝 `frontend/rust-lib/flowy-database2/src/event_handler.rs` (+3 -1) 📝 `frontend/rust-lib/flowy-database2/src/services/database/database_editor.rs` (+27 -1) 📝 `frontend/rust-lib/flowy-database2/src/services/database_view/view_editor.rs` (+6 -4) </details> ### 📄 Description Closes: #3409 ### Feature Preview https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/32bc5066-8df9-4ef8-87ff-f50dcfe92c46 #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] 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. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>