[PR #3658] [MERGED] feat: reminder improvements #5810

New issue

Closed

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

mirror commented

2026-03-23 22:20:25 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3658
Author: @Xazin
Created: 10/9/2023
Status: ✅ Merged
Merged: 10/12/2023
Merged by: @LucasXu0

Base: main ← Head: feat/notification-upcoming

📝 Commits (6)

944a47c feat: enable past and upcoming reminders
4dec2ce feat: group by date
2156484 fix: lints and analyzer
bbaf011 chore: localization
fdc59d8 Merge remote-tracking branch 'origin/main' into feat/notification-upcoming
3dc2849 fix: resolve comments

📊 Changes

26 files changed (+785 additions, -154 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/accessory/cell_accessory.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/block_action_button.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/align_toolbar_item/align_toolbar_item.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/link_to_page_widget.dart (+2 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/more/more_button.dart (+1 -1)
➕ frontend/appflowy_flutter/lib/user/application/notification_filter/notification_filter_bloc.dart (+65 -0)
📝 frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart (+52 -37)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_top_menu.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_user.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_button.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_dialog.dart (+396 -75)
➕ frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_group.dart (+58 -0)
➕ frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_grouped_view.dart (+65 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_item.dart (+18 -10)
➕ frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_view.dart (+59 -0)
➕ frontend/appflowy_flutter/lib/workspace/presentation/notifications/notifications_hub_empty.dart (+20 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_file_customize_location_view.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_user_view.dart (+1 -1)

...and 6 more files

📄 Description

Relates: #3596

Feature Preview

Will add a preview soon.

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/3658 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 10/9/2023 **Status:** ✅ Merged **Merged:** 10/12/2023 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `feat/notification-upcoming` --- ### 📝 Commits (6) - [`944a47c`](https://github.com/AppFlowy-IO/AppFlowy/commit/944a47c0879cae5f3e50f2a4c165a361ff43631c) feat: enable past and upcoming reminders - [`4dec2ce`](https://github.com/AppFlowy-IO/AppFlowy/commit/4dec2ce86e4817a99db6f505e760feec7dc98912) feat: group by date - [`2156484`](https://github.com/AppFlowy-IO/AppFlowy/commit/21564845ff3ff6a68518e53d09c01babee7b9491) fix: lints and analyzer - [`bbaf011`](https://github.com/AppFlowy-IO/AppFlowy/commit/bbaf0115b6397f86165f483f15fec5da1ffa3c73) chore: localization - [`fdc59d8`](https://github.com/AppFlowy-IO/AppFlowy/commit/fdc59d86fceca61e829bcedbcec2286d0e4074d3) Merge remote-tracking branch 'origin/main' into feat/notification-upcoming - [`3dc2849`](https://github.com/AppFlowy-IO/AppFlowy/commit/3dc284984a2f1eb32a955ec3c92b3bbadbdeb20a) fix: resolve comments ### 📊 Changes **26 files changed** (+785 additions, -154 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/accessory/cell_accessory.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/block_action_button.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/align_toolbar_item/align_toolbar_item.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/link_to_page_widget.dart` (+2 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/more/more_button.dart` (+1 -1) ➕ `frontend/appflowy_flutter/lib/user/application/notification_filter/notification_filter_bloc.dart` (+65 -0) 📝 `frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart` (+52 -37) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_top_menu.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_user.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_button.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_dialog.dart` (+396 -75) ➕ `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_group.dart` (+58 -0) ➕ `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_grouped_view.dart` (+65 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_item.dart` (+18 -10) ➕ `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_view.dart` (+59 -0) ➕ `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notifications_hub_empty.dart` (+20 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_file_customize_location_view.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/settings/widgets/settings_user_view.dart` (+1 -1) _...and 6 more files_ </details> ### 📄 Description Relates: #3596 ### Feature Preview Will add a preview soon. #### 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. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>