Added more logging events

WindowsAppCommunity · May 1, 2022 · 6e75dfe · 6e75dfe
1 parent 5737d57
commit 6e75dfe
Show file tree

Hide file tree

Showing 5 changed files with 50 additions and 5 deletions.
diff --git a/src/Quarrel.ViewModels/Services/APIs/GitHubService/GitHubService.cs b/src/Quarrel.ViewModels/Services/APIs/GitHubService/GitHubService.cs
@@ -73,8 +73,9 @@ public GitHubService(IAnalyticsService analyticsService)
                 var user = await _gitHubApiService.GetUserAsync(contributor.Name);
                 return new BindableDeveloper(user, contributor);
             }
-            catch
+            catch (Exception ex)
             {
+                _analyticsService.Log(LoggedEvent.GitHubRequestFailed, ("Endpoint", "GetDeveloper"), ("Exception", ex.Message));
                 return null;
             }
         }

diff --git a/src/Quarrel.ViewModels/Services/Analytics/Enums/LoggedEvent.cs b/src/Quarrel.ViewModels/Services/Analytics/Enums/LoggedEvent.cs
@@ -9,6 +9,20 @@ namespace Quarrel.Services.Analytics.Enums
     /// </summary>
     public enum LoggedEvent
     {
+        #region App
+        /// <summary>
+        /// A subpage opened.
+        /// </summary>
+        [StringValue("SubPage Opened")]
+        SubPageOpened,
+
+        /// <summary>
+        /// A message was sent.
+        /// </summary>
+        [StringValue("Message Sent")]
+        MessageSent,
+        #endregion
+
         #region Login
         /// <summary>
         /// The app attempted to login.
@@ -29,12 +43,16 @@ public enum LoggedEvent
         LoginFailed,
         #endregion
 
+        #region API
+
         /// <summary>
         /// Encountered an issue in an http request, but it was handled.
         /// </summary>
         [StringValue("Http Exception Handled")]
         HttpExceptionHandled,
 
+        #endregion
+
         #region Gateway
         /// <summary>
         /// The gateway encountered a known event.
@@ -87,6 +105,12 @@ public enum LoggedEvent
         [StringValue("GitHub Request Failed")]
         GitHubRequestFailed,
 
+        /// <summary>
+        /// A request to the Discord Status API failed.
+        /// </summary>
+        [StringValue("Discord Status Request Failed")]
+        DiscordStatusRequestFailed,
+
         #endregion
 
         #region AppServiceConnection

diff --git a/src/Quarrel.ViewModels/Services/Discord/DiscordService.Methods.cs b/src/Quarrel.ViewModels/Services/Discord/DiscordService.Methods.cs
@@ -14,6 +14,7 @@
 using Quarrel.Client.Models.Messages;
 using Quarrel.Client.Models.Settings;
 using Quarrel.Client.Models.Users;
+using Quarrel.Services.Analytics.Enums;
 using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
@@ -180,6 +181,8 @@ public async Task<Message[]> GetChannelMessagesAsync(IBindableMessageChannel cha
         /// <inheritdoc/>
         public void SendMessage(ulong channelId, string content)
         {
+            _analyticsService.Log(LoggedEvent.MessageSent);
+
             _quarrelClient.SendMessage(channelId, content);
         }
     }

diff --git a/src/Quarrel.ViewModels/ViewModels/SubPages/DiscordStatus/DiscordStatusViewModel.cs b/src/Quarrel.ViewModels/ViewModels/SubPages/DiscordStatus/DiscordStatusViewModel.cs
@@ -4,8 +4,11 @@
 using Discord.API.Status.Models;
 using Discord.API.Status.Rest;
 using Microsoft.Toolkit.Mvvm.ComponentModel;
+using Quarrel.Services.Analytics;
+using Quarrel.Services.Analytics.Enums;
 using Quarrel.ViewModels.SubPages.DiscordStatus.Enums;
 using Quarrel.ViewModels.SubPages.DiscordStatus.Models;
+using Refit;
 using System;
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
@@ -17,6 +20,8 @@ namespace Quarrel.ViewModels.SubPages.DiscordStatus
     /// </summary>
     public partial class DiscordStatusViewModel : ObservableObject
     {
+        private readonly IAnalyticsService _analyticsService;
+
         private Index? _status;
         private IStatusService? _statusService;
 
@@ -29,8 +34,9 @@ public partial class DiscordStatusViewModel : ObservableObject
         /// <summary>
         /// Initializes a new instance of the <see cref="DiscordStatusViewModel"/> class.
         /// </summary>
-        public DiscordStatusViewModel()
+        public DiscordStatusViewModel(IAnalyticsService analyticsService)
         {
+            _analyticsService = analyticsService;
             _state = DiscordStatusState.Loading;
             SetupAndLoad();
         }
@@ -110,8 +116,9 @@ private async void SetupAndLoad()
             {
                 Status = await _statusService.GetStatus();
             }
-            catch
+            catch (ApiException ex)
             {
+                _analyticsService.Log(LoggedEvent.DiscordStatusRequestFailed, ("Endpoint", "GetStatus"), ("Exception", ex.Message));
             }
 
             // Has Data
@@ -196,7 +203,10 @@ public async void ShowMetrics(string duration)
                 Guard.IsNotNull(_statusService);
                 metrics = await _statusService.GetMetrics(duration);
             }
-            catch { }
+            catch (Exception ex)
+            {
+                _analyticsService.Log(LoggedEvent.DiscordStatusRequestFailed, ("Endpoint", "GetMetrics"), ("Exception", ex.Message));
+            }
 
             if (metrics != null && metrics.Metrics != null && metrics.Metrics.Length > 0)
             {

diff --git a/src/Quarrel.ViewModels/ViewModels/SubPages/Host/SubPageHostViewModel.cs b/src/Quarrel.ViewModels/ViewModels/SubPages/Host/SubPageHostViewModel.cs
@@ -6,6 +6,8 @@
 using Microsoft.Toolkit.Mvvm.Input;
 using Microsoft.Toolkit.Mvvm.Messaging;
 using Quarrel.Messages.Navigation.SubPages;
+using Quarrel.Services.Analytics;
+using Quarrel.Services.Analytics.Enums;
 using System;
 using System.Collections.Generic;
 
@@ -16,6 +18,7 @@ namespace Quarrel.ViewModels.SubPages.Host
     /// </summary>
     public partial class SubPageHostViewModel : ObservableRecipient
     {
+        private readonly IAnalyticsService _analyticsService;
         private readonly IMessenger _messenger;
         private readonly Stack<object> _navStack;
 
@@ -24,8 +27,9 @@ public partial class SubPageHostViewModel : ObservableRecipient
         /// <summary>
         /// Initializes a new instance of the <see cref="SubPageHostViewModel"/> class.
         /// </summary>
-        public SubPageHostViewModel(IMessenger messenger)
+        public SubPageHostViewModel(IAnalyticsService analyticsService, IMessenger messenger)
         {
+            _analyticsService = analyticsService;
             _messenger = messenger;
             _navStack = new Stack<object>();
 
@@ -69,6 +73,9 @@ private void NavigateToSubPage(Type viewModelType)
                 _navStack.Push(ContentViewModel);
             }
 
+            _analyticsService.Log(LoggedEvent.SubPageOpened,
+                ("Type", viewModelType.Name));
+
             ContentViewModel = viewModel;
         }