diff --git a/Hammer/CommandModules/BanCommand.cs b/Hammer/CommandModules/BanCommand.cs index 295f94c..23c0dd1 100644 --- a/Hammer/CommandModules/BanCommand.cs +++ b/Hammer/CommandModules/BanCommand.cs @@ -1,4 +1,4 @@ -using System; +using System; using System.Threading.Tasks; using BrackeysBot.API.Extensions; using BrackeysBot.Core.API; @@ -42,9 +42,9 @@ public BanCommand(BanService banService, RuleService ruleService) [SlashCommand("ban", "Temporarily or permanently bans a user.", false)] [SlashRequireGuild] public async Task BanAsync(InteractionContext context, - [Option("user", "The user to mute")] DiscordUser user, - [Option("reason", "The reason for the mute")] string? reason = null, - [Option("duration", "The duration of the mute")] string? durationRaw = null, + [Option("user", "The user to ban.")] DiscordUser user, + [Option("reason", "The reason for the ban.")] string? reason = null, + [Option("duration", "The duration of the ban.")] string? durationRaw = null, [Option("rule", "The rule which was broken."), Autocomplete(typeof(RuleAutocompleteProvider))] long? ruleBroken = null) { await context.DeferAsync(true).ConfigureAwait(false); diff --git a/Hammer/CommandModules/Infractions/InfractionCommand.Copy.cs b/Hammer/CommandModules/Infractions/InfractionCommand.Copy.cs index 9edd665..8254c82 100644 --- a/Hammer/CommandModules/Infractions/InfractionCommand.Copy.cs +++ b/Hammer/CommandModules/Infractions/InfractionCommand.Copy.cs @@ -15,7 +15,7 @@ internal sealed partial class InfractionCommand [SlashRequireGuild] public async Task CopyAsync(InteractionContext context, [Option("source", "The user whose infractions to copy.")] DiscordUser source, - [Option("destination", "The user whose infractions to copy.")] DiscordUser destination) + [Option("destination", "The user who will acquire the copied infractions.")] DiscordUser destination) { await context.DeferAsync(true).ConfigureAwait(false); diff --git a/Hammer/CommandModules/Infractions/InfractionCommand.Delete.cs b/Hammer/CommandModules/Infractions/InfractionCommand.Delete.cs index 6ab2773..ab74652 100644 --- a/Hammer/CommandModules/Infractions/InfractionCommand.Delete.cs +++ b/Hammer/CommandModules/Infractions/InfractionCommand.Delete.cs @@ -10,21 +10,21 @@ namespace Hammer.CommandModules.Infractions; internal sealed partial class InfractionCommand { - [SlashCommand("delete", "Deletes an infraction with the specified ID.", false)] + [SlashCommand("delete", "Deletes an infraction.", false)] [SlashRequireGuild] public async Task DeleteAsync(InteractionContext context, - [Autocomplete(typeof(InfractionAutocompleteProvider))] [Option("id", "The ID of the infraction to delete.")] - long id) + [Autocomplete(typeof(InfractionAutocompleteProvider))] [Option("infraction", "The infraction to delete.")] + long infractionId) { await context.DeferAsync(true).ConfigureAwait(false); var embed = new DiscordEmbedBuilder(); - Infraction? infraction = _infractionService.GetInfraction(id); + Infraction? infraction = _infractionService.GetInfraction(infractionId); if (infraction is null) { embed.WithColor(0xFF0000); embed.WithTitle("Infraction not found"); - embed.WithDescription($"The infraction with the ID `{id}` was not found."); + embed.WithDescription($"The infraction with the ID `{infractionId}` was not found."); } else { diff --git a/Hammer/CommandModules/Infractions/InfractionCommand.Move.cs b/Hammer/CommandModules/Infractions/InfractionCommand.Move.cs index 1abd702..94c94d9 100644 --- a/Hammer/CommandModules/Infractions/InfractionCommand.Move.cs +++ b/Hammer/CommandModules/Infractions/InfractionCommand.Move.cs @@ -13,8 +13,8 @@ internal sealed partial class InfractionCommand [SlashCommand("move", "Moves all infractions from one user to another.", false)] [SlashRequireGuild] public async Task MoveAsync(InteractionContext context, - [Option("source", "The user whose infractions to copy.")] DiscordUser source, - [Option("destination", "The user whose infractions to copy.")] DiscordUser destination) + [Option("source", "The user whose infractions to move.")] DiscordUser source, + [Option("destination", "The user who will acquire the moved infractions.")] DiscordUser destination) { await context.DeferAsync(true).ConfigureAwait(false); diff --git a/Hammer/CommandModules/UnbanCommandModule.cs b/Hammer/CommandModules/UnbanCommandModule.cs index 1d0c5e8..825d846 100644 --- a/Hammer/CommandModules/UnbanCommandModule.cs +++ b/Hammer/CommandModules/UnbanCommandModule.cs @@ -1,4 +1,4 @@ -using System; +using System; using System.Threading.Tasks; using BrackeysBot.API.Extensions; using BrackeysBot.Core.API; @@ -34,8 +34,7 @@ public UnbanCommandModule(BanService banService) [Description("Unbans a user.")] [RequirePermissionLevel(PermissionLevel.Administrator)] public async Task UnbanCommandAsync(CommandContext context, [Description("The ID of the user to unban.")] ulong userId, - [Description("The reason for the ban revocation."), RemainingText] - string? reason = null) + [Description("The reason for the ban revocation."), RemainingText] string? reason = null) { await context.AcknowledgeAsync().ConfigureAwait(false); @@ -63,8 +62,7 @@ public async Task UnbanCommandAsync(CommandContext context, [Description("The ID [Description("Unbans a user.")] [RequirePermissionLevel(PermissionLevel.Administrator)] public async Task UnbanCommandAsync(CommandContext context, [Description("The user to unban.")] DiscordUser user, - [Description("The reason for the ban revocation."), RemainingText] - string? reason = null) + [Description("The reason for the ban revocation."), RemainingText] string? reason = null) { await context.AcknowledgeAsync().ConfigureAwait(false);