chore(notification): add todo comments with follow up tasks

Author: rafaeltonholo

app-common/src/main/kotlin/net/thunderbird/app/common/notification/LegacyNotificationIdFactory.kt

@@ -4,6 +4,7 @@ import com.fsck.k9.notification.NotificationIds
 import net.thunderbird.feature.notification.api.NotificationId
 import net.thunderbird.feature.notification.api.NotificationIdFactory
 
+// TODO(#9416): Migrate logic from NotificationIds to NotificationIdFactory
 class LegacyNotificationIdFactory : NotificationIdFactory {
     override fun next(
         accountNumber: Int,

feature/notification/api/src/commonMain/kotlin/net/thunderbird/feature/notification/api/NotificationGroup.kt

@@ -3,7 +3,7 @@ package net.thunderbird.feature.notification.api
 import net.thunderbird.core.common.io.KmpParcelable
 import net.thunderbird.core.common.io.KmpParcelize
 
-// TODO: Properly handle notification groups, adding summary, etc.
+// TODO(9419): Properly handle notification groups, adding summary, etc.
 @KmpParcelize
 data class NotificationGroup(
     val key: NotificationGroupKey,

feature/notification/api/src/commonMain/kotlin/net/thunderbird/feature/notification/api/NotificationIdFactory.kt

@@ -1,5 +1,6 @@
 package net.thunderbird.feature.notification.api
 
+// TODO(#9416): Migrate logic from NotificationIds to NotificationIdFactory
 interface NotificationIdFactory {
     fun next(accountNumber: Int, offset: Int = 0): NotificationId
 }

feature/notification/impl/src/commonMain/kotlin/net/thunderbird/feature/notification/impl/command/InAppNotificationCommand.kt

@@ -31,5 +31,7 @@ internal class InAppNotificationCommand(
 
     // TODO(#9392): Verify if the app is on foreground. IF it isn't, then should fail
     //  executing the command
+    // TODO(#9420): If the app is on background and the severity is Fatal or Critical, we should
+    //  let the command execute, but store it in a database instead of triggering the show notification logic.
     private fun canExecuteCommand(): Boolean = true
 }

legacy/core/src/main/java/com/fsck/k9/notification/NotificationIds.kt

@@ -2,6 +2,7 @@ package com.fsck.k9.notification
 
 import net.thunderbird.core.android.account.LegacyAccount
 
+// TODO(#9416): Migrate logic from NotificationIds to NotificationIdFactory
 object NotificationIds {
     const val PUSH_NOTIFICATION_ID = 1
     const val BACKGROUND_WORK_NOTIFICATION_ID = 2