Merge pull request #1510 from BCSDLab/feature/1507-add-notification-screen

[Feature] 알림 화면 구현

Author: kongwoojin

core/navigation/src/main/java/in/koreatech/koin/core/navigation/Navigator.kt

@@ -18,6 +18,11 @@ interface Navigator {
         vararg args: Pair<String, Any?> // Extra IDs
     ): Intent
 
+    fun navigateToScheme(
+        context: Context,
+        extraUrl: String
+    ): Intent
+
     fun navigateToSignIn(
         context: Context,
         redirectUrl: String? = null

data/src/main/java/in/koreatech/koin/data/dao/NotificationDao.kt

@@ -8,6 +8,7 @@ import androidx.room.Transaction
 import `in`.koreatech.koin.data.constant.DBConstant
 import `in`.koreatech.koin.data.entity.NotificationEntity
 import java.time.LocalDateTime
+import kotlinx.coroutines.flow.Flow
 
 @Suppress("Detekt.TooManyFunctions")
 @Dao
@@ -21,6 +22,9 @@ interface NotificationDao {
     @Query("SELECT * FROM ${DBConstant.NOTIFICATION}")
     suspend fun getNotifications(): List<NotificationEntity>
 
+    @Query("SELECT * FROM ${DBConstant.NOTIFICATION}")
+    fun getNotificationsFlow(): Flow<List<NotificationEntity>>
+
     @Query("SELECT * FROM ${DBConstant.NOTIFICATION} WHERE originUrl = :url")
     suspend fun getNotificationByUrl(url: String): NotificationEntity
 

data/src/main/java/in/koreatech/koin/data/repository/NotificationRepositoryImpl.kt

@@ -16,6 +16,8 @@ import `in`.koreatech.koin.domain.model.notification.SubscribesType
 import `in`.koreatech.koin.domain.repository.NotificationRepository
 import javax.inject.Inject
 import kotlin.collections.map
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.map
 import retrofit2.HttpException
 
 @Suppress("Detekt.TooManyFunctions")
@@ -122,6 +124,11 @@ class NotificationRepositoryImpl @Inject constructor(
         }
     }
 
+    override fun getNotificationsFlowFromLocal(): Flow<List<Notification>> {
+        return notificationLocalDataSource.getNotificationsFlow()
+            .map { list -> list.map { it.toNotification() } }
+    }
+
     override suspend fun deleteNotificationFromLocal(id: Int): Result<Unit> {
         return suspendRunCatching {
             notificationLocalDataSource.deleteNotification(id)

data/src/main/java/in/koreatech/koin/data/source/local/NotificationLocalDataSource.kt

@@ -4,6 +4,7 @@ import `in`.koreatech.koin.data.dao.NotificationDao
 import `in`.koreatech.koin.data.entity.NotificationEntity
 import java.time.LocalDateTime
 import javax.inject.Inject
+import kotlinx.coroutines.flow.Flow
 
 class NotificationLocalDataSource @Inject constructor(
     private val notificationDao: NotificationDao
@@ -20,6 +21,10 @@ class NotificationLocalDataSource @Inject constructor(
         return notificationDao.getNotifications()
     }
 
+    fun getNotificationsFlow(): Flow<List<NotificationEntity>> {
+        return notificationDao.getNotificationsFlow()
+    }
+
     suspend fun updateNotificationReadByUrl(url: String, isRead: Boolean): NotificationEntity {
         return notificationDao.updateReadByUrlAndReturn(url, isRead)
     }

domain/src/main/java/in/koreatech/koin/domain/repository/NotificationRepository.kt

@@ -4,6 +4,7 @@ import `in`.koreatech.koin.domain.model.notification.Notification
 import `in`.koreatech.koin.domain.model.notification.NotificationPermissionInfo
 import `in`.koreatech.koin.domain.model.notification.SubscribesDetailType
 import `in`.koreatech.koin.domain.model.notification.SubscribesType
+import kotlinx.coroutines.flow.Flow
 
 @Suppress("Detekt.TooManyFunctions")
 interface NotificationRepository {
@@ -29,6 +30,8 @@ interface NotificationRepository {
 
     suspend fun getNotificationsFromLocal(): Result<List<Notification>>
 
+    fun getNotificationsFlowFromLocal(): Flow<List<Notification>>
+
     suspend fun deleteNotificationFromLocal(id: Int): Result<Unit>
 
     suspend fun deleteNotificationsFromLocal(ids: List<Int>): Result<Unit>

domain/src/main/java/in/koreatech/koin/domain/usecase/notification/DeleteNotificationsUseCase.kt

@@ -0,0 +1,12 @@
+package `in`.koreatech.koin.domain.usecase.notification
+
+import `in`.koreatech.koin.domain.repository.NotificationRepository
+import javax.inject.Inject
+
+class DeleteNotificationsUseCase @Inject constructor(
+    private val notificationRepository: NotificationRepository
+) {
+    suspend operator fun invoke(ids: List<Int>): Result<Unit> {
+        return notificationRepository.deleteNotificationsFromLocal(ids)
+    }
+}

domain/src/main/java/in/koreatech/koin/domain/usecase/notification/GetNotificationsFlowUseCase.kt

@@ -0,0 +1,14 @@
+package `in`.koreatech.koin.domain.usecase.notification
+
+import `in`.koreatech.koin.domain.model.notification.Notification
+import `in`.koreatech.koin.domain.repository.NotificationRepository
+import javax.inject.Inject
+import kotlinx.coroutines.flow.Flow
+
+class GetNotificationsFlowUseCase @Inject constructor(
+    private val notificationRepository: NotificationRepository
+) {
+    operator fun invoke(): Flow<List<Notification>> {
+        return notificationRepository.getNotificationsFlowFromLocal()
+    }
+}

domain/src/main/java/in/koreatech/koin/domain/usecase/notification/UpdateNotificationsReadByIdUseCase.kt

@@ -0,0 +1,28 @@
+package `in`.koreatech.koin.domain.usecase.notification
+
+import `in`.koreatech.koin.domain.model.notification.Notification
+import `in`.koreatech.koin.domain.repository.NotificationRepository
+import javax.inject.Inject
+import kotlinx.coroutines.async
+import kotlinx.coroutines.awaitAll
+import kotlinx.coroutines.coroutineScope
+
+class UpdateNotificationsReadByIdUseCase @Inject constructor(
+    private val notificationRepository: NotificationRepository
+) {
+    suspend operator fun invoke(
+        ids: List<Int>,
+        isRead: Boolean
+    ): Result<List<Notification>> = coroutineScope {
+        val results = ids.map { id ->
+            async {
+                notificationRepository.updateNotificationReadById(
+                    id = id,
+                    isRead = isRead
+                )
+            }
+        }.awaitAll()
+
+        Result.success(results.mapNotNull { it.getOrNull() })
+    }
+}

feature/home/src/main/java/in/koreatech/koin/feature/home/HomeScreen.kt

@@ -7,15 +7,19 @@ import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.Spacer
+import androidx.compose.foundation.layout.WindowInsets
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
+import androidx.compose.foundation.layout.statusBars
 import androidx.compose.foundation.layout.width
 import androidx.compose.foundation.rememberScrollState
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.foundation.verticalScroll
 import androidx.compose.material3.Icon
+import androidx.compose.material3.Scaffold
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
@@ -57,26 +61,38 @@ import org.orbitmvi.orbit.compose.collectAsState
 
 @Composable
 fun HomeScreen(
-    viewModel: HomeViewModel = hiltViewModel()
+    viewModel: HomeViewModel = hiltViewModel(),
+    navigateToNotification: () -> Unit = {}
 ) {
     val uiState by viewModel.collectAsState()
 
-    HomeScreen(
-        modifier = Modifier.background(Color(0xFFF8F8FA)),
-        uiState = uiState
-    )
+    Scaffold(
+        modifier = Modifier.fillMaxSize(),
+        contentWindowInsets = WindowInsets.statusBars
+    ) { paddingValues ->
+        HomeScreen(
+            modifier = Modifier
+                .fillMaxSize()
+                .background(Color(0xFFF8F8FA))
+                .padding(paddingValues),
+            uiState = uiState,
+            navigateToNotification = navigateToNotification
+        )
+    }
 }
 
 @Composable
 private fun HomeScreen(
     uiState: HomeState,
-    modifier: Modifier = Modifier
+    modifier: Modifier = Modifier,
+    navigateToNotification: () -> Unit = {}
 ) {
     Column(
         modifier = modifier.verticalScroll(rememberScrollState())
     ) {
         HeaderSection(
-            isNewNotificationReceived = uiState.isNewNotificationReceived
+            isNewNotificationReceived = uiState.isNewNotificationReceived,
+            onNotificationClick = navigateToNotification
         )
         InfoSection(
             username = uiState.username,
@@ -107,7 +123,8 @@ private fun HeaderSection(
     Row(
         modifier = modifier
             .fillMaxWidth()
-            .padding(vertical = 8.dp, horizontal = 24.dp)
+            .padding(vertical = 8.dp, horizontal = 24.dp),
+        verticalAlignment = Alignment.CenterVertically
     ) {
         Image(
             imageVector = ImageVector.vectorResource(R.drawable.ic_bcsd_symbol),

feature/notification/AGENTS.md

@@ -0,0 +1,19 @@
+# Feature Notification Module - AGENTS.md
+
+`feature/notification` is a Compose-first Orbit module for Notification list screen.
+
+## Keep In Mind
+
+- Use the existing Compose + Orbit patterns for screens and ViewModels.
+- Keep notification-specific flows inside this module.
+- Use `rememberNavigator()` and shared navigation helpers for hand-offs back to `koin` Activities.
+- Reuse shared onboarding, analytics, and design-system components before adding module-local alternatives.
+- Keep Notification-specific UI state and validation in the feature layer; ViewModels still call use cases rather than repositories directly.
+
+## Read First
+
+- Root `AGENTS.md`
+- `core/AGENTS.md`
+- `core/navigation/AGENTS.md`
+- `core/designsystem/AGENTS.md`
+- `core/onboarding/AGENTS.md`