[PR #5860] [MERGED] chore: add more logs in reminder bloc #7053

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5860
Author: @LucasXu0
Created: 8/2/2024
Status: ✅ Merged
Merged: 8/8/2024
Merged by: @LucasXu0

Base: main ← Head: add_logs_to_reminder

📝 Commits (10+)

9d0bb8f chore: add more logs in reminder bloc
75fbda3 Revert "chore: add more logs in reminder bloc"
9307a8d chore: add more logs in reminder bloc
6bb26b7 Merge branch 'main' into add_logs_to_reminder
1ad9bf8 Merge branch 'main' into add_logs_to_reminder
bbb760c fix: unable to view reminders on Desktop
d3753e1 fix: force refresh reminders
fed43ec chore: fix flutter analyze
97533a6 feat: support database reminder on Mobile
71c0653 chore: remove referenced database padding

📊 Changes

18 files changed (+218 additions, -106 deletions)

View changed files

📝 frontend/appflowy_flutter/ios/Podfile.lock (+2 -2)
📝 frontend/appflowy_flutter/lib/mobile/application/notification/notification_reminder_bloc.dart (+29 -6)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/mobile_view_page.dart (+9 -2)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/notifications/widgets/shared.dart (+50 -10)
📝 frontend/appflowy_flutter/lib/plugins/database/board/presentation/widgets/board_hidden_groups.dart (+5 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_page.dart (+11 -7)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculations_row.dart (+7 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/footer/grid_footer.dart (+6 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/row.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/database_view_widget.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/shared/google_fonts_extension.dart (+1 -6)
📝 frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart (+64 -19)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_dialog.dart (+6 -9)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_button.dart (+11 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_item.dart (+11 -27)
📝 frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_view.dart (+1 -1)

📄 Description

Feature Preview

Add logs to address the reminder missing issue.

PR Checklist

Add more logs in reminder block to address the status issue.

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/5860 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 8/2/2024 **Status:** ✅ Merged **Merged:** 8/8/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `add_logs_to_reminder` --- ### 📝 Commits (10+) - [`9d0bb8f`](https://github.com/AppFlowy-IO/AppFlowy/commit/9d0bb8fb2963d24cc572754c2f9411cebfa69f98) chore: add more logs in reminder bloc - [`75fbda3`](https://github.com/AppFlowy-IO/AppFlowy/commit/75fbda3418ca27e7f9a25449f6255d9be11008ca) Revert "chore: add more logs in reminder bloc" - [`9307a8d`](https://github.com/AppFlowy-IO/AppFlowy/commit/9307a8d89aa611c0f4f67b5b3cd134b221d5fcd8) chore: add more logs in reminder bloc - [`6bb26b7`](https://github.com/AppFlowy-IO/AppFlowy/commit/6bb26b7da6933f367b6ec2ad55bdded9b4d8b7b9) Merge branch 'main' into add_logs_to_reminder - [`1ad9bf8`](https://github.com/AppFlowy-IO/AppFlowy/commit/1ad9bf81c8bef1b527155b3856a6a452ef08fe15) Merge branch 'main' into add_logs_to_reminder - [`bbb760c`](https://github.com/AppFlowy-IO/AppFlowy/commit/bbb760ce967b9d42113a7202b5c54051d3ee8c99) fix: unable to view reminders on Desktop - [`d3753e1`](https://github.com/AppFlowy-IO/AppFlowy/commit/d3753e177b19f51add7ae3a574bfde625d5c432f) fix: force refresh reminders - [`fed43ec`](https://github.com/AppFlowy-IO/AppFlowy/commit/fed43ecdfc2bf8add4d0a25a5849cbcbd954e5ad) chore: fix flutter analyze - [`97533a6`](https://github.com/AppFlowy-IO/AppFlowy/commit/97533a675d9df7e665285b92ca3b71d1c02d34f9) feat: support database reminder on Mobile - [`71c0653`](https://github.com/AppFlowy-IO/AppFlowy/commit/71c0653d4850b68db36209ad0666bf025c1baf2a) chore: remove referenced database padding ### 📊 Changes **18 files changed** (+218 additions, -106 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/ios/Podfile.lock` (+2 -2) 📝 `frontend/appflowy_flutter/lib/mobile/application/notification/notification_reminder_bloc.dart` (+29 -6) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/mobile_view_page.dart` (+9 -2) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_page.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/notifications/widgets/shared.dart` (+50 -10) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/presentation/widgets/board_hidden_groups.dart` (+5 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_page.dart` (+11 -7) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/calculations/calculations_row.dart` (+7 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/footer/grid_footer.dart` (+6 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/row.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/database_view_widget.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/shared/google_fonts_extension.dart` (+1 -6) 📝 `frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart` (+64 -19) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/notification_dialog.dart` (+6 -9) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_button.dart` (+11 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_item.dart` (+11 -27) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/notifications/widgets/notification_view.dart` (+1 -1) </details> ### 📄 Description   ### Feature Preview Add logs to address the reminder missing issue.  ---  #### PR Checklist Add more logs in reminder block to address the status issue. - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>