Merge pull request #1924 from Infomaniak/sentry-notifications

Remove `reason` in `sendFailedNotification()` to dissociate Sentries
Infomaniak · Jun 19, 2024 · ee60d35 · ee60d35
2 parents c93bb2f + 2eba450
commit ee60d35
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/app/src/main/java/com/infomaniak/mail/utils/SentryDebug.kt b/app/src/main/java/com/infomaniak/mail/utils/SentryDebug.kt
@@ -148,15 +148,14 @@ object SentryDebug {
 
             scope.level = sentryLevel
 
-            scope.setTag("reason", reason)
             scope.setExtra("userId", "${userId?.toString()}")
             scope.setExtra("currentUserId", "[${AccountUtils.currentUserId}]")
             scope.setExtra("mailboxId", "${mailboxId?.toString()}")
             scope.setExtra("mailbox.email", "[${mailbox?.email}]")
             scope.setExtra("currentMailboxEmail", "[${AccountUtils.currentMailboxEmail}]")
             scope.setExtra("messageUid", "$messageUid")
 
-            val message = "We received a Notification, but we failed to show it"
+            val message = "Failed Notif : $reason"
 
             throwable?.let {
                 scope.setExtra("message", message)