Added more comments

Author: Avid29

src/Quarrel.Client/QuarrelClient.Methods.cs

@@ -26,12 +26,19 @@ public partial class QuarrelClient
             return CurrentUser;
         }
 
+        /// <summary>
+        /// Gets a user by id.
+        /// </summary>
+        /// <param name="id">The id of the user to get.</param>
         public User? GetUser(ulong id)
         {
             _userMap.TryGetValue(id, out var user);
             return user;
         }
 
+        /// <summary>
+        /// Gets the client settings.
+        /// </summary>
         public Settings? GetSettings()
         {
             return _settings;
@@ -166,6 +173,11 @@ public IPrivateChannel[] GetPrivateChannels()
             return privateChannels;
         }
 
+        /// <summary>
+        /// Sends a message.
+        /// </summary>
+        /// <param name="channelId">The channel to send the message in.</param>
+        /// <param name="content">The content of the message.</param>
         public async Task SendMessage(ulong channelId, string content)
         {
             Guard.IsNotNull(_channelService, nameof(_channelService));
@@ -174,12 +186,21 @@ public async Task SendMessage(ulong channelId, string content)
             await MakeRefitRequest(() => _channelService.CreateMessage(channelId, message));
         }
 
+        /// <summary>
+        /// Deletes a message.
+        /// </summary>
+        /// <param name="channelId">The id of channel the message belongs to.</param>
+        /// <param name="messageId">The id of the message to delete.</param>
         public async Task DeleteMessage(ulong channelId, ulong messageId)
         {
             Guard.IsNotNull(_channelService, nameof(_channelService));
             await MakeRefitRequest(() => _channelService.DeleteMessage(channelId, messageId));
         }
 
+        /// <summary>
+        /// Updates the user's online status.
+        /// </summary>
+        /// <param name="status">The new online status to set.</param>
         public async Task UpdateStatus(UserStatus status)
         {
             Guard.IsNotNull(_gateway, nameof(_gateway));

src/Quarrel.ViewModels/Services/Discord/DiscordService.Events.cs

@@ -10,7 +10,7 @@ namespace Quarrel.Services.Discord
 {
     public partial class DiscordService
     {
-        public void RegisterChannelEvents()
+        private void RegisterChannelEvents()
         {
             _quarrelClient.MessageCreated += OnMessageCreate;
             _quarrelClient.MessageUpdated += OnMessageUpdated;

src/Quarrel.ViewModels/Services/Discord/IDiscordService.cs

@@ -78,10 +78,24 @@ public interface IDiscordService
 
         BindableGuildMember? GetGuildMember(ulong userId, ulong guildId);
 
+        /// <summary>
+        /// Sends a message.
+        /// </summary>
+        /// <param name="channelId">The id of the channel to send the message in.</param>
+        /// <param name="content">The content of the message.</param>
         Task SendMessage(ulong channelId, string content);
 
+        /// <summary>
+        /// Deletes a message.
+        /// </summary>
+        /// <param name="channelId">The id of the channel the message is in.</param>
+        /// <param name="messageId">The id of the message to delete.</param>
         Task DeleteMessage(ulong channelId, ulong messageId);
 
+        /// <summary>
+        /// Updates the user's online status.
+        /// </summary>
+        /// <param name="status">The new online status to set.</param>
         Task SetStatus(UserStatus status);
     }
 }

src/Quarrel.ViewModels/ViewModels/Panels/MessageBoxViewModel.cs

@@ -10,15 +10,21 @@
 
 namespace Quarrel.ViewModels.Panels
 {
+    /// <summary>
+    /// The view model for the message box.
+    /// </summary>
     public class MessageBoxViewModel : ObservableRecipient
     {
         private readonly IMessenger _messenger;
         private readonly IDiscordService _discordService;
         private readonly IDispatcherService _dispatcherService;
 
         private ulong? _channelId;
-        private string _draftText;
+        private string? _draftText;
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="MessageBoxViewModel"/> class.
+        /// </summary>
         public MessageBoxViewModel(IMessenger messenger, IDiscordService discordService, IDispatcherService dispatcherService)
         {
             _messenger = messenger;
@@ -33,25 +39,34 @@ public MessageBoxViewModel(IMessenger messenger, IDiscordService discordService,
             });
         }
 
+        /// <summary>
+        /// Gets a command that sends the drafted message.
+        /// </summary>
         public RelayCommand SendMessageCommand { get; }
 
-        public string DraftText
+        /// <summary>
+        /// Gets or sets the drafted text in the message box.
+        /// </summary>
+        public string? DraftText
         {
             get => _draftText;
             set => SetProperty(ref _draftText, value);
         }
 
+        /// <summary>
+        /// Gets the current channel id.
+        /// </summary>
         public ulong? ChannelId
         {
             get => _channelId;
-            set => SetProperty(ref _channelId, value);
+            private set => SetProperty(ref _channelId, value);
         }
 
-        public void SendMessage()
+        private void SendMessage()
         {
-            if (ChannelId.HasValue)
+            if (ChannelId.HasValue && !string.IsNullOrEmpty(DraftText))
             {
-                _discordService.SendMessage(ChannelId.Value, DraftText);
+                _discordService.SendMessage(ChannelId.Value, DraftText!);
                 DraftText = string.Empty;
             }
         }