Merge pull request #35 from Novack/WorkaroundForIssue30

Temporary workaround for issue #30
skiff-org · Dec 8, 2023 · 1f57076 · 1f57076
2 parents d98d661 + 7cdb366
commit 1f57076
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/skiffWindowsApp/Skiff Desktop/MessageProcessor.cs b/skiffWindowsApp/Skiff Desktop/MessageProcessor.cs
@@ -18,7 +18,7 @@ internal class MessageProcessor
     {
         private MainWindow _mainWindow;
         private NotificationsController _notificationsController;
-
+        private List<string> _consumedMessages = new();
 
         public MessageProcessor(MainWindow mainWindow, NotificationsController notificationsController)
         {
@@ -48,6 +48,10 @@ internal void CoreWebView2_WebMessageReceived(object sender, CoreWebView2WebMess
                         var notificationsPayload = JsonSerializer.Deserialize<NotificationDataWrapper>(receivedMessage.Data.ToString());
                         foreach (var notification in notificationsPayload.NotificationData)
                         {
+                            if (_consumedMessages.Contains(notification.EmailId))
+                                continue;
+
+                            _consumedMessages.Add(notification.EmailId);
                             Debug.WriteLine($"Displaying toast with title: {notification.Title} and body: {notification.Body}");
                             _notificationsController.ShowToastNotification(notification.Title, notification.Body, notification.ThreadId);
                         }
@@ -122,6 +126,8 @@ public class NotificationItem
             public string Body { get; set; }
             [JsonPropertyName("threadID")]
             public string ThreadId { get; set; }
+            [JsonPropertyName("emailID")]
+            public string EmailId { get; set; }
         }
 
         public class UnreadCountDataWrapper