From d74d43e4c8872e71b003ab74c04fb7f9dd6eb6c9 Mon Sep 17 00:00:00 2001 From: Kevin Boulongne Date: Mon, 10 Jun 2024 11:54:16 +0200 Subject: [PATCH] Remove Sentry about "No new Message" notification --- .../infomaniak/mail/utils/FetchMessagesManager.kt | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/com/infomaniak/mail/utils/FetchMessagesManager.kt b/app/src/main/java/com/infomaniak/mail/utils/FetchMessagesManager.kt index 58113044950..dc1753e6270 100644 --- a/app/src/main/java/com/infomaniak/mail/utils/FetchMessagesManager.kt +++ b/app/src/main/java/com/infomaniak/mail/utils/FetchMessagesManager.kt @@ -152,16 +152,12 @@ class FetchMessagesManager @Inject constructor( if (mailboxGroupNotifications.size == 1) cancel(mailbox.notificationGroupId) } + /** + * When we fetched Messages, we didn't find any new Message. + * It means we already got them all when we received a previous notification. + * We can leave safely. + */ if (threadsWithNewMessages.isEmpty()) { - SentryDebug.sendFailedNotification( - reason = "No new Message", - sentryLevel = SentryLevel.WARNING, - userId = userId, - mailboxId = mailbox.mailboxId, - messageUid = sentryMessageUid, - mailbox = mailbox, - ) - realm.close() return }