Feature: Send message via user dialog

app/src/main/java/de/tum/informatics/www1/artemis/native_app/android/ui/MainActivity.kt

@@ -109,13 +109,17 @@ class MainActivity : AppCompatActivity(),
         val visibleMetisContextManager = object :
             VisibleMetisContextManager {
             override fun registerMetisContext(metisContext: VisibleMetisContext) {
-                visibleMetisContexts.value = visibleMetisContexts.value + metisContext
+                visibleMetisContexts.value += metisContext
 
                 cancelCommunicationNotifications(metisContext)
             }
 
             override fun unregisterMetisContext(metisContext: VisibleMetisContext) {
-                visibleMetisContexts.value = visibleMetisContexts.value - metisContext
+                visibleMetisContexts.value -= metisContext
+            }
+
+            override fun getRegisteredMetisContexts(): List<VisibleMetisContext> {
+                return visibleMetisContexts.value
             }
         }
 

core/data/src/debug/kotlin/de/tum/informatics/www1/artemis/native_app/core/data/AccountDataServiceStub.kt → core/data-test/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/core/data/test/AccountDataServiceStub.kt

@@ -1,5 +1,6 @@
-package de.tum.informatics.www1.artemis.native_app.core.data
+package de.tum.informatics.www1.artemis.native_app.core.data.test
 
+import de.tum.informatics.www1.artemis.native_app.core.data.NetworkResponse
 import de.tum.informatics.www1.artemis.native_app.core.data.service.network.AccountDataService
 import de.tum.informatics.www1.artemis.native_app.core.model.account.Account
 

feature/core-modules-test/src/test/kotlin/de/tum/informatics/www1/artemis/native_app/feature/coremodulestest/AppModuleTest.kt

@@ -8,6 +8,7 @@ import de.tum.informatics.www1.artemis.native_app.android.appModule
 import de.tum.informatics.www1.artemis.native_app.core.common.CurrentActivityListener
 import de.tum.informatics.www1.artemis.native_app.core.common.test.UnitTest
 import de.tum.informatics.www1.artemis.native_app.feature.metis.shared.content.StandalonePostId
+import de.tum.informatics.www1.artemis.native_app.feature.metis.shared.content.UserIdentifier
 import de.tum.informatics.www1.artemis.native_app.feature.quiz.QuizType
 import org.junit.Test
 import org.junit.experimental.categories.Category
@@ -32,6 +33,7 @@ class AppModuleTest {
                 SavedStateHandle::class,
                 Boolean::class,
                 Long::class,
+                UserIdentifier::class,
                 StandalonePostId::class,
                 Application::class,
                 CurrentActivityListener::class,

feature/course-view/build.gradle.kts

@@ -17,6 +17,7 @@ dependencies {
     implementation(libs.kotlinx.datetime)
     implementation(libs.placeholder.material)
     implementation(project(":core:device-test"))
+    debugImplementation(project(":core:data-test"))
 
     testImplementation(project(":feature:login"))
     testImplementation(project(":feature:login-test"))

feature/course-view/src/debug/kotlin/de/tum/informatics/www1/artemis/native_app/feature/courseview/CourseViewScreenshots.kt

@@ -19,6 +19,8 @@ import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.CourseVi
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.CourseUiScreen
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_CONVERSATION_ID
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_POST_ID
+import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_USERNAME
+import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_USER_ID
 import org.koin.core.context.startKoin
 import org.koin.dsl.module
 
@@ -54,8 +56,9 @@ fun `Course View - Exercise List`() {
         CourseUiScreen(
             modifier = Modifier.fillMaxSize(),
             viewModel = courseViewModel,
-            username = "",
             courseId = 0L,
+            username = DEFAULT_USERNAME,
+            userId = DEFAULT_USER_ID,
             conversationId = DEFAULT_CONVERSATION_ID,
             postId = DEFAULT_POST_ID,
             onNavigateToExercise = {},

feature/course-view/src/debug/kotlin/de/tum/informatics/www1/artemis/native_app/feature/courseview/MessagingScreenshots.kt

@@ -10,8 +10,8 @@ import androidx.compose.runtime.remember
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.text.input.TextFieldValue
-import de.tum.informatics.www1.artemis.native_app.core.data.AccountDataServiceStub
 import de.tum.informatics.www1.artemis.native_app.core.data.DataState
+import de.tum.informatics.www1.artemis.native_app.core.data.test.AccountDataServiceStub
 import de.tum.informatics.www1.artemis.native_app.core.datastore.AccountServiceStub
 import de.tum.informatics.www1.artemis.native_app.core.datastore.ServerConfigurationServiceStub
 import de.tum.informatics.www1.artemis.native_app.core.device.NetworkStatusProviderStub
@@ -197,11 +197,9 @@ fun `Metis - Conversation Channel`() {
     ).reversed()
 
     val visibleMetisContextManagerStub = object : VisibleMetisContextManager {
-        override fun registerMetisContext(metisContext: VisibleMetisContext) =
-            Unit
-
-        override fun unregisterMetisContext(metisContext: VisibleMetisContext) =
-            Unit
+        override fun registerMetisContext(metisContext: VisibleMetisContext) = Unit
+        override fun unregisterMetisContext(metisContext: VisibleMetisContext) = Unit
+        override fun getRegisteredMetisContexts(): List<VisibleMetisContext> = emptyList()
     }
 
     // TODO: Provide artemis image provider

feature/course-view/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/courseview/ui/course_overview/CourseUiScreen.kt

@@ -46,11 +46,13 @@ import de.tum.informatics.www1.artemis.native_app.feature.courseview.R
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.CourseViewModel
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.LectureListUi
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.exercise_list.ExerciseListUi
+import de.tum.informatics.www1.artemis.native_app.feature.metis.IgnoreCustomBackHandling
 import de.tum.informatics.www1.artemis.native_app.feature.metis.NavigateToUserConversation
 import de.tum.informatics.www1.artemis.native_app.feature.metis.NothingOpened
 import de.tum.informatics.www1.artemis.native_app.feature.metis.OpenedConversation
 import de.tum.informatics.www1.artemis.native_app.feature.metis.OpenedThread
 import de.tum.informatics.www1.artemis.native_app.feature.metis.shared.content.StandalonePostId
+import de.tum.informatics.www1.artemis.native_app.feature.metis.shared.content.UserIdentifier
 import de.tum.informatics.www1.artemis.native_app.feature.metis.ui.ConversationFacadeUi
 import kotlinx.serialization.Serializable
 import org.koin.androidx.compose.koinViewModel
@@ -62,13 +64,16 @@ internal const val TAB_COMMUNICATION = 2
 
 internal const val DEFAULT_CONVERSATION_ID = -1L
 internal const val DEFAULT_POST_ID = -1L
+internal const val DEFAULT_USERNAME = ""
+internal const val DEFAULT_USER_ID = -1L
 
 @Serializable
 private data class CourseUiScreen(
     val courseId: Long,
     val conversationId: Long = DEFAULT_CONVERSATION_ID,
     val postId: Long = DEFAULT_POST_ID,
-    val username: String = ""
+    val username: String = DEFAULT_USERNAME,
+    val userId: Long = DEFAULT_USER_ID
 )
 
 fun NavController.navigateToCourse(courseId: Long, builder: NavOptionsBuilder.() -> Unit) {
@@ -92,7 +97,8 @@ fun NavGraphBuilder.course(
             generateLinks("courses/{courseId}") +
             generateLinks("courses/{courseId}/exercises") +
             generateLinks("courses/{courseId}/messages?conversationId={conversationId}") +
-            generateLinks("courses/{courseId}/messages?username={username}")
+            generateLinks("courses/{courseId}/messages?username={username}") +
+            generateLinks("courses/{courseId}/messages?userId={userId}")
     animatedComposable<CourseUiScreen>(
         deepLinks = deepLinks
     ) { backStackEntry ->
@@ -102,6 +108,7 @@ fun NavGraphBuilder.course(
         val conversationId = route.conversationId
         val postId = route.postId
         val username = route.username
+        val userId = route.userId
 
         CourseUiScreen(
             modifier = Modifier.fillMaxSize(),
@@ -110,6 +117,7 @@ fun NavGraphBuilder.course(
             conversationId = conversationId,
             postId = postId,
             username = username,
+            userId = userId,
             onNavigateToExercise = onNavigateToExercise,
             onNavigateToTextExerciseParticipation = onNavigateToTextExerciseParticipation,
             onNavigateToExerciseResultView = onNavigateToExerciseResultView,
@@ -135,6 +143,7 @@ fun CourseUiScreen(
     conversationId: Long,
     postId: Long,
     username: String,
+    userId: Long,
     onNavigateToExercise: (exerciseId: Long) -> Unit,
     onNavigateToTextExerciseParticipation: (exerciseId: Long, participationId: Long) -> Unit,
     onNavigateToExerciseResultView: (exerciseId: Long) -> Unit,
@@ -152,6 +161,7 @@ fun CourseUiScreen(
         conversationId = conversationId,
         courseDataState = courseDataState,
         username = username,
+        userId = userId,
         onNavigateBack = onNavigateBack,
         weeklyExercisesDataState = weeklyExercisesDataState,
         onNavigateToExercise = onNavigateToExercise,
@@ -182,6 +192,7 @@ internal fun CourseUiScreen(
     conversationId: Long,
     postId: Long,
     username: String,
+    userId: Long,
     courseDataState: DataState<Course>,
     weeklyExercisesDataState: DataState<List<GroupedByWeek<Exercise>>>,
     weeklyLecturesDataState: DataState<List<GroupedByWeek<Lecture>>>,
@@ -197,7 +208,9 @@ internal fun CourseUiScreen(
 ) {
     var selectedTabIndex by rememberSaveable(conversationId) {
         val initialTab = when {
-            conversationId != DEFAULT_CONVERSATION_ID || username.isNotBlank() -> TAB_COMMUNICATION
+            conversationId != DEFAULT_CONVERSATION_ID -> TAB_COMMUNICATION
+            username != DEFAULT_USERNAME -> TAB_COMMUNICATION
+            userId != DEFAULT_USER_ID -> TAB_COMMUNICATION
             else -> TAB_EXERCISES
         }
 
@@ -256,19 +269,28 @@ internal fun CourseUiScreen(
                 val initialConfiguration = remember(conversationId, postId) {
                     when {
                         conversationId != DEFAULT_CONVERSATION_ID && postId != DEFAULT_POST_ID -> OpenedConversation(
-                            conversationId,
-                            OpenedThread(
-                                conversationId,
+                            _prevConfiguration = IgnoreCustomBackHandling,
+                            conversationId = conversationId,
+                            openedThread = OpenedThread(
                                 StandalonePostId.ServerSideId(postId)
                             )
                         )
 
                         conversationId != DEFAULT_CONVERSATION_ID -> OpenedConversation(
-                            conversationId,
-                            null
+                            _prevConfiguration = IgnoreCustomBackHandling,
+                            conversationId = conversationId,
+                            openedThread = null
                         )
 
-                        username.isNotBlank() -> NavigateToUserConversation(username)
+                        username != DEFAULT_USERNAME -> NavigateToUserConversation(
+                            _prevConfiguration = IgnoreCustomBackHandling,
+                            userIdentifier = UserIdentifier.Username(username)
+                        )
+
+                        userId != DEFAULT_USER_ID -> NavigateToUserConversation(
+                            _prevConfiguration = IgnoreCustomBackHandling,
+                            userIdentifier = UserIdentifier.UserId(userId)
+                        )
 
                         else -> NothingOpened
                     }

feature/course-view/src/test/kotlin/de/tum/informatics/www1/artemis/native_app/feature/courseview/BaseCourseTest.kt

@@ -12,6 +12,8 @@ import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.CourseVi
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.CourseUiScreen
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_CONVERSATION_ID
 import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_POST_ID
+import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_USERNAME
+import de.tum.informatics.www1.artemis.native_app.feature.courseview.ui.course_overview.DEFAULT_USER_ID
 import de.tum.informatics.www1.artemis.native_app.feature.login.loginModule
 import de.tum.informatics.www1.artemis.native_app.feature.login.test.getAdminAccessToken
 import de.tum.informatics.www1.artemis.native_app.feature.login.test.performTestLogin
@@ -61,7 +63,8 @@ abstract class BaseCourseTest : BaseComposeTest() {
                 courseId = course.id!!,
                 conversationId = DEFAULT_CONVERSATION_ID,
                 postId = DEFAULT_POST_ID,
-                username = "",
+                username = DEFAULT_USERNAME,
+                userId = DEFAULT_USER_ID,
                 onNavigateToExercise = {},
                 onNavigateToExerciseResultView = {},
                 onNavigateToTextExerciseParticipation = { _, _ -> },

feature/metis/conversation/src/main/kotlin/de/tum/informatics/www1/artemis/native_app/feature/metis/conversation/ui/chatlist/MetisChatList.kt

@@ -145,15 +145,6 @@ fun MetisChatList(
     title: String,
     onFileSelected: (Uri) -> Unit
 ) {
-    val context = LocalContext.current
-
-    // TODO: https://github.com/ls1intum/artemis-android/issues/213
-//    val navigateToChat = { userId: Long ->
-//        val chatLink = "artemis://courses/$courseId/messages?userId=$userId"
-//        val intent = Intent(Intent.ACTION_VIEW, Uri.parse(chatLink))
-//        context.startActivity(intent)
-//    }
-
     MetisReplyHandler(
         initialReplyTextProvider = initialReplyTextProvider,
         onCreatePost = onCreatePost,