Skip to content

Commit 28f2176

Browse files
authored
Merge pull request #938 from allthingslinux/reminder-hotfix
fix(remindme.py): fix issue where old reminders from the old system w…
2 parents b12d079 + 2437be7 commit 28f2176

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

tux/cogs/utility/remindme.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,15 @@ async def on_ready(self) -> None:
7272
dt_now = datetime.datetime.now(datetime.UTC)
7373

7474
for reminder in reminders:
75+
# hotfix for an issue where old reminders from the old system would all send at once
76+
if reminder.reminder_sent:
77+
try:
78+
await self.db.reminder.delete_reminder_by_id(reminder.reminder_id)
79+
except Exception as e:
80+
logger.error(f"Failed to delete reminder: {e}")
81+
82+
continue
83+
7584
seconds = (reminder.reminder_expires_at - dt_now).total_seconds()
7685

7786
if seconds <= 0:

0 commit comments

Comments
 (0)