[PR #4735] [MERGED] fix: launch review 0.5.0 #6413

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

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4735
Author: @Xazin
Created: 2/25/2024
Status: Merged
Merged: 2/25/2024
Merged by: @Xazin

Base: mainHead: launch_review/0.5.0


📝 Commits (9)

  • eb0070a fix: top bar buttons slow respond
  • 380cbb2 chore: merge branch 'upstream/main'
  • 99ac620 fix: fav button responsive
  • bcca409 fix: add favorite button to database views
  • c96a986 fix: add overflow to calculation
  • a809bca fix: minor workaround for showing number format prefix
  • 7271749 fix: clear calculation on empty field cells
  • 35f94bb fix: average calculation should not have def value
  • 9806f34 fix: cargo fmt

📊 Changes

9 files changed (+84 additions, -72 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculate_cell.dart (+18 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/tab_bar/tab_bar_view.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/share_button.dart (+8 -6)
📝 frontend/appflowy_flutter/lib/plugins/document/document.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/share/share_button.dart (+8 -6)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/favorite_button.dart (+26 -31)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/view_title_bar.dart (+7 -6)
📝 frontend/rust-lib/flowy-database2/src/services/calculations/controller.rs (+11 -17)
📝 frontend/rust-lib/flowy-database2/src/services/calculations/service.rs (+1 -1)

📄 Description

Closes: #4730
Closes: #4729

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/4735 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 2/25/2024 **Status:** ✅ Merged **Merged:** 2/25/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `launch_review/0.5.0` --- ### 📝 Commits (9) - [`eb0070a`](https://github.com/AppFlowy-IO/AppFlowy/commit/eb0070a564d6935c320182d056220aafd0d27bbf) fix: top bar buttons slow respond - [`380cbb2`](https://github.com/AppFlowy-IO/AppFlowy/commit/380cbb20b566181e8f366f1717d32c23565bbe23) chore: merge branch 'upstream/main' - [`99ac620`](https://github.com/AppFlowy-IO/AppFlowy/commit/99ac6201ca6ffc22ef35ac6431946bcbc4e4add9) fix: fav button responsive - [`bcca409`](https://github.com/AppFlowy-IO/AppFlowy/commit/bcca4096a3ff5d06cf5ca3d5bd1744b05d1a7e30) fix: add favorite button to database views - [`c96a986`](https://github.com/AppFlowy-IO/AppFlowy/commit/c96a9862dd4a0e2ca10482e25a5b5505ac7a0535) fix: add overflow to calculation - [`a809bca`](https://github.com/AppFlowy-IO/AppFlowy/commit/a809bca20573434cd34af4bf78bcbdd3be25a62a) fix: minor workaround for showing number format prefix - [`7271749`](https://github.com/AppFlowy-IO/AppFlowy/commit/7271749d5d78ebb0078703e0ac0a40ca69b0d967) fix: clear calculation on empty field cells - [`35f94bb`](https://github.com/AppFlowy-IO/AppFlowy/commit/35f94bb1c03c46bae9cf0eb3c69b6c3bb81a552d) fix: average calculation should not have def value - [`9806f34`](https://github.com/AppFlowy-IO/AppFlowy/commit/9806f34545c0ff3f93495a441837f719e07d1679) fix: cargo fmt ### 📊 Changes **9 files changed** (+84 additions, -72 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculate_cell.dart` (+18 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/tab_bar/tab_bar_view.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/share_button.dart` (+8 -6) 📝 `frontend/appflowy_flutter/lib/plugins/document/document.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/share/share_button.dart` (+8 -6) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/favorite_button.dart` (+26 -31) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/view_title_bar.dart` (+7 -6) 📝 `frontend/rust-lib/flowy-database2/src/services/calculations/controller.rs` (+11 -17) 📝 `frontend/rust-lib/flowy-database2/src/services/calculations/service.rs` (+1 -1) </details> ### 📄 Description Closes: #4730 Closes: #4729 #### 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. - [x] 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>
mirror 2026-03-23 22:23:06 +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#6413
No description provided.