[PR #7869] [MERGED] feat: remove reminder in the same event #8174

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7869
Author: @LucasXu0
Created: 4/30/2025
Status: ✅ Merged
Merged: 4/30/2025
Merged by: @LucasXu0

Base: main ← Head: remove_reminders

📝 Commits (1)

c8e2b02 feat: remove reminder in the same event

📊 Changes

5 files changed (+65 additions, -50 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/date_cell_editor_bloc.dart (+5 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/date_transaction_handler.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_date_block.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/user/application/password/password_http_service.dart (+6 -3)
📝 frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart (+51 -40)

📄 Description

Feature Preview

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.

Summary by Sourcery

Refactor and improve the reminder management system in the AppFlowy application, focusing on more efficient reminder handling and removal.

New Features:

Add support for removing multiple reminders at once

Bug Fixes:

Fix potential issues with reminder removal across different components

Enhancements:

Optimize reminder filtering by caching views
Improve bulk reminder removal process
Simplify reminder availability checking

_{🔄 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/7869 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 4/30/2025 **Status:** ✅ Merged **Merged:** 4/30/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `remove_reminders` --- ### 📝 Commits (1) - [`c8e2b02`](https://github.com/AppFlowy-IO/AppFlowy/commit/c8e2b02689c1d5b5741912d6ed762f2f129d368b) feat: remove reminder in the same event ### 📊 Changes **5 files changed** (+65 additions, -50 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/date_cell_editor_bloc.dart` (+5 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/date_transaction_handler.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mention/mention_date_block.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/user/application/password/password_http_service.dart` (+6 -3) 📝 `frontend/appflowy_flutter/lib/user/application/reminder/reminder_bloc.dart` (+51 -40) </details> ### 📄 Description   ### Feature Preview  ---  #### 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. ## Summary by Sourcery Refactor and improve the reminder management system in the AppFlowy application, focusing on more efficient reminder handling and removal. New Features: - Add support for removing multiple reminders at once Bug Fixes: - Fix potential issues with reminder removal across different components Enhancements: - Optimize reminder filtering by caching views - Improve bulk reminder removal process - Simplify reminder availability checking --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>