fix: .dmcmd will now correctly block commands in dms, not globally

change: timely will no longer require guild context, as dmcmd .timely will do the same thing

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 Mostly based on [keepachangelog](https://keepachangelog.com/en/1.0.0/) except date format. a-c-f-r-o
 
-## [5.3.0] - 07.12.2024
+## [5.3.0] - 10.12.2024
 
 ## Added
 
@@ -35,10 +35,12 @@ Mostly based on [keepachangelog](https://keepachangelog.com/en/1.0.0/) except da
 - `.sclr show` will now show hex code of the current color
 - Queueing a song will now restart the playback if the queue is on the last track and stopped (there were no more tracks
   to play)
+- `.translate` will now use 2 embeds instead of 1
 
 ## Fixed
 
 - .setstream and .setactivity will now pause .ropl (rotating statuses)
+- Fixed `.sar ex` help description
 
 ## Removed
 

src/NadekoBot/Modules/Administration/Honeypot/HoneyPotService.cs

@@ -71,7 +71,7 @@ public sealed class HoneyPotService : IHoneyPotService, IReadyExecutor, IExecNoC
                 try
                 {
                     Log.Information("Honeypot caught user {User} [{UserId}]", user, user.Id);
-                    await user.BanAsync(pruneDays: 1);
+                    await user.BanAsync(pruneDays: 1, reason: "Honeypot");
                     await user.Guild.RemoveBanAsync(user.Id);
                 }
                 catch (Exception e)

src/NadekoBot/Modules/Gambling/BetStatsCommands.cs

@@ -149,7 +149,10 @@ public partial class Gambling
                                    ?? (await _userService.GetUserAsync(x.UserId))?.Username
                                    ?? x.UserId.ToString();
 
-                        outputItems.Add(new WinLbStat(i + 1 + (page * 10), user, x.Game, x.MaxWin));
+                        if (user.StartsWith("??"))
+                            user = x.UserId.ToString();
+
+                        outputItems.Add(new WinLbStat(i + 1 + (page * 9), user, x.Game, x.MaxWin));
                     }
 
                     return outputItems;
@@ -166,11 +169,12 @@ public partial class Gambling
             await Response()
                   .Paginated()
                   .PageItems(p => GetCachedWinLbAsync(p))
-                  .PageSize(10)
+                  .PageSize(9)
                   .Page((items, curPage) =>
                   {
                       var eb = CreateEmbed()
-                          .WithOkColor();
+                               .WithTitle(GetText(strs.winlb))
+                               .WithOkColor();
 
                       if (items.Count == 0)
                       {
@@ -182,7 +186,8 @@ public partial class Gambling
                       {
                           var item = items[i];
                           eb.AddField($"#{item.Rank} {item.User}",
-                              $"{N(item.MaxWin)}\n`{item.Game.ToString().ToLower()}`");
+                              $"{N(item.MaxWin)}\n`{item.Game.ToString().ToLower()}`",
+                              true);
                       }
 
                       return eb;

src/NadekoBot/Modules/Gambling/Gambling.cs

@@ -135,7 +135,6 @@ public partial class Gambling : GamblingModule<GamblingService>
                 });
 
     [Cmd]
-    [RequireContext(ContextType.Guild)]
     public async Task Timely()
     {
         var val = Config.Timely.Amount;

src/NadekoBot/Modules/Gambling/UserBetStatsService.cs

@@ -60,8 +60,8 @@ public sealed class UserBetStatsService : INService
         await using var ctx = _db.GetDbContext();
         return await ctx.GetTable<UserBetStats>()
                         .OrderByDescending(x => x.MaxWin)
-                        .Skip(page * 10)
-                        .Take(10)
+                        .Skip(page * 9)
+                        .Take(9)
                         .ToArrayAsyncLinqToDB();
     }
 }

src/NadekoBot/Modules/Permissions/GlobalPermissions/GlobalPermissionService.cs

@@ -92,13 +92,13 @@ public class GlobalPermissionService : IExecPreCommand, INService
         {
             if (priv)
             {
-                if (bs.Blocked.Commands.Add(commandName))
+                if (bs.DmBlocked.Commands.Add(commandName))
                 {
                     added = true;
                 }
                 else
                 {
-                    bs.Blocked.Commands.Remove(commandName);
+                    bs.DmBlocked.Commands.Remove(commandName);
                     added = false;
                 }
 

src/NadekoBot/Modules/Searches/Searches.cs

@@ -417,15 +417,24 @@ public partial class Searches : NadekoModule<SearchesService>
     {
         usr ??= (IGuildUser)ctx.User;
 
-        var bannerUrl = usr.GetGuildBannerUrl();
+        var bannerUrl = usr.GetGuildBannerUrl(size: 2048);
+
+        if (bannerUrl is null)
+        {
+            await Response()
+                  .Error(strs.no_banner)
+                  .SendAsync();
+
+            return;
+        }
 
         await Response()
               .Embed(
                   CreateEmbed()
                       .WithOkColor()
-                      .AddField("Username", usr.ToString())
-                      .AddField("Banner Url", bannerUrl)
-                      .WithThumbnailUrl(bannerUrl))
+                      .AddField("Username", usr.ToString(), true)
+                      .AddField("Banner Url", bannerUrl, true)
+                      .WithImageUrl(bannerUrl))
               .SendAsync();
     }
 

src/NadekoBot/Modules/Utility/GuildColorsCommands.cs

@@ -13,9 +13,9 @@ public partial class Utility
         public async Task ServerColorsShow()
         {
             var colors = _service.GetColors(ctx.Guild.Id);
-            var okHex = colors?.Ok?.RawValue.ToString("X8");
-            var warnHex = colors?.Warn?.RawValue.ToString("X8");
-            var errHex = colors?.Error?.RawValue.ToString("X8");
+            var okHex = colors?.Ok?.RawValue.ToString("x6");
+            var warnHex = colors?.Warn?.RawValue.ToString("x6");
+            var errHex = colors?.Error?.RawValue.ToString("x6");
             EmbedBuilder[] ebs =
             [
                 CreateEmbed()

src/NadekoBot/Modules/Utility/Utility.cs

@@ -186,7 +186,7 @@ public partial class Utility : NadekoModule
 
                   return CreateEmbed()
                                 .WithOkColor()
-                                .WithTitle(GetText(strs.inrole_list(roleName, roleUsers.Count)))
+                                .WithTitle(GetText(strs.inrole_list(role?.GetIconUrl() + roleName, roleUsers.Count)))
                                 .WithDescription(string.Join("\n", pageUsers));
               })
               .SendAsync();

src/NadekoBot/data/commandlist.json

@@ -961,6 +961,56 @@
         "MuteMembers Server Permission"
       ]
     },
+    {
+      "Aliases": [
+        ".notify",
+        ".nfy"
+      ],
+      "Description": "Sends a message to the current channel once the specified event occurs.\nProvide no parameters to see all available events.",
+      "Usage": [
+        ".notify levelup Congratulations to user %user.name% for reaching level %event.level%"
+      ],
+      "Submodule": "NotifyCommands",
+      "Module": "Administration",
+      "Options": null,
+      "Requirements": [
+        "Bot Owner Only"
+      ]
+    },
+    {
+      "Aliases": [
+        ".notifylist",
+        ".notifyl"
+      ],
+      "Description": "Lists all active notifications in this server.",
+      "Usage": [
+        ".notifylist"
+      ],
+      "Submodule": "NotifyCommands",
+      "Module": "Administration",
+      "Options": null,
+      "Requirements": [
+        "Bot Owner Only"
+      ]
+    },
+    {
+      "Aliases": [
+        ".notifyclear",
+        ".notifyremove",
+        ".notifyrm",
+        ".notifclr"
+      ],
+      "Description": "Removes the specified notify event.",
+      "Usage": [
+        ".notifyclear levelup"
+      ],
+      "Submodule": "NotifyCommands",
+      "Module": "Administration",
+      "Options": null,
+      "Requirements": [
+        "Bot Owner Only"
+      ]
+    },
     {
       "Aliases": [
         ".dpo"
@@ -1523,6 +1573,22 @@
         "Administrator Server Permission"
       ]
     },
+    {
+      "Aliases": [
+        ".temprole"
+      ],
+      "Description": "Grants a user a temporary role for the specified number of time.\nThe role must exist and be lower in the role hierarchy than your highest role.",
+      "Usage": [
+        ".temprole 15m @User Jail",
+        ".temprole 7d @Newbie Trial Member"
+      ],
+      "Submodule": "RoleCommands",
+      "Module": "Administration",
+      "Options": null,
+      "Requirements": [
+        "Administrator Server Permission"
+      ]
+    },
     {
       "Aliases": [
         ".iam"
@@ -1648,9 +1714,9 @@
         ".sar excl",
         ".sar tesar"
       ],
-      "Description": "Toggles whether self-assigned roles are exclusive. While enabled, users can only have one self-assignable role per group.",
+      "Description": "Toggles the sar group as exclusive.\nWhile enabled, users can only have one self-assignable role from that group.",
       "Usage": [
-        ".sar exclusive"
+        ".sar exclusive 1"
       ],
       "Submodule": "sar",
       "Module": "Administration",
@@ -3274,6 +3340,21 @@
       "Options": null,
       "Requirements": []
     },
+    {
+      "Aliases": [
+        ".winlb",
+        ".wins"
+      ],
+      "Description": "Shows the biggest wins leaderboard",
+      "Usage": [
+        ".winlb",
+        ".winlb 5"
+      ],
+      "Submodule": "BetStatsCommands",
+      "Module": "Gambling",
+      "Options": null,
+      "Requirements": []
+    },
     {
       "Aliases": [
         ".gamblestats",
@@ -3915,6 +3996,20 @@
       "Options": null,
       "Requirements": []
     },
+    {
+      "Aliases": [
+        ".minesweeper",
+        ".mw"
+      ],
+      "Description": "Creates a spoiler-based minesweeper mini game.\nYou may specify the number of mines.",
+      "Usage": [
+        ".minesweeper 15"
+      ],
+      "Submodule": "Games",
+      "Module": "Games",
+      "Options": null,
+      "Requirements": []
+    },
     {
       "Aliases": [
         ".acrophobia",
@@ -5522,6 +5617,38 @@
         "Bot Owner Only"
       ]
     },
+    {
+      "Aliases": [
+        ".dmmodule",
+        ".dmmod"
+      ],
+      "Description": "Toggles whether a module can be used in DMs.",
+      "Usage": [
+        ".dmmodule Gambling"
+      ],
+      "Submodule": "GlobalPermissionCommands",
+      "Module": "Permissions",
+      "Options": null,
+      "Requirements": [
+        "Bot Owner Only"
+      ]
+    },
+    {
+      "Aliases": [
+        ".dmcommand",
+        ".dmcmd"
+      ],
+      "Description": "Toggles whether a command can be used in DMs.",
+      "Usage": [
+        ".dmcommand .stats"
+      ],
+      "Submodule": "GlobalPermissionCommands",
+      "Module": "Permissions",
+      "Options": null,
+      "Requirements": [
+        "Bot Owner Only"
+      ]
+    },
     {
       "Aliases": [
         ".resetperms"
@@ -5791,6 +5918,19 @@
       "Options": null,
       "Requirements": []
     },
+    {
+      "Aliases": [
+        ".banner"
+      ],
+      "Description": "Shows a mentioned person's banner.",
+      "Usage": [
+        ".banner @Someone"
+      ],
+      "Submodule": "Searches",
+      "Module": "Searches",
+      "Options": null,
+      "Requirements": []
+    },
     {
       "Aliases": [
         ".wikia",
@@ -7778,21 +7918,6 @@
       "Options": null,
       "Requirements": []
     },
-    {
-      "Aliases": [
-        ".xpnotify",
-        ".xpn"
-      ],
-      "Description": "Sets how the bot should notify you when you get a `server` or `global` level. This is a personal setting and affects only how you receive Global or Server level-up notifications. You can set `dm` (for the bot to send you a direct message), `channel` (to get notified in the channel you sent the last message in) or `none` to disable.",
-      "Usage": [
-        ".xpnotify global dm",
-        ".xpnotify server channel"
-      ],
-      "Submodule": "Xp",
-      "Module": "Xp",
-      "Options": null,
-      "Requirements": []
-    },
     {
       "Aliases": [
         ".xpexclude",
@@ -7854,6 +7979,21 @@
       "Options": null,
       "Requirements": []
     },
+    {
+      "Aliases": [
+        ".xplevelset"
+      ],
+      "Description": "Sets the level of the user you specify.",
+      "Usage": [
+        ".xplevelset 10 @User"
+      ],
+      "Submodule": "Xp",
+      "Module": "Xp",
+      "Options": null,
+      "Requirements": [
+        "Administrator Server Permission"
+      ]
+    },
     {
       "Aliases": [
         ".xpadd"

src/NadekoBot/data/strings/responses/responses.en-US.json

@@ -1157,5 +1157,7 @@
   "notify_desc_protection": "Triggers when antialt, antispam or antiraid is triggered.",
   "notify_desc_addrolerew": "Triggers when a user gets a role as a reward for reaching a level (xprew).",
   "notify_desc_removerolerew": "Triggers when a user loses a role as a reward for reaching a level (xprew).",
-  "notify_desc_not_found": "No description found for this notify event. Please report this."
+  "notify_desc_not_found": "No description found for this notify event. Please report this.",
+  "winlb": "Biggest Wins Leaderboard",
+  "no_banner": "No banner set."
 }