Address Review

- Convert `ids` to a set to remove duplicates
- Limit the amount of reminders you can delete at once to 5 in order to prevent API spam

1 files changed, 6 insertions, 2 deletions

diff --git a/bot/exts/utils/reminders.py b/bot/exts/utils/reminders.py
index 51fdd7873..4f7e1c255 100644
--- a/bot/exts/utils/reminders.py
+++ b/bot/exts/utils/reminders.py
@@ -465,9 +465,13 @@ class Reminders(Cog):
 
     @remind_group.command("delete", aliases=("remove", "cancel"))
     async def delete_reminder(self, ctx: Context, ids: Greedy[int]) -> None:
-        """Delete one of your active reminders."""
+        """Delete up to (and including) 5 of your active reminders."""
+        if len(ids) > 5:
+            await send_denial(ctx, "You can only delete a maximum of 5 reminders at once.")
+            return
+
         deleted_ids = []
-        for id_ in ids:
+        for id_ in set(ids):
             try:
                 reminder_deleted = await self._delete_reminder(ctx, id_)
             except LockedResourceError: