Report a bug change

Author: domonkosadam

libs/horizon/src/main/java/com/instructure/horizon/features/account/AccountScreen.kt

@@ -38,6 +38,7 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.clip
 import androidx.compose.ui.platform.LocalContext
+import androidx.compose.ui.platform.LocalUriHandler
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.unit.dp
 import androidx.navigation.NavController
@@ -156,6 +157,7 @@ private fun AccountContentScreen(state: AccountUiState, navController: NavContro
 
 @Composable
 private fun AccountItem(item: AccountItemState, navController: NavController, onLogout: () -> Unit, switchExperience: () -> Unit, modifier: Modifier = Modifier) {
+    val uriHandler = LocalUriHandler.current
     Row(
         verticalAlignment = Alignment.CenterVertically,
         modifier = modifier
@@ -165,7 +167,9 @@ private fun AccountItem(item: AccountItemState, navController: NavController, on
                 when (item.type) {
                     is AccountItemType.Open -> navController.navigate(item.type.route.route)
 
-                    is AccountItemType.OpenExternal -> navController.navigate(item.type.route.route)
+                    is AccountItemType.OpenExternal -> {
+                        uriHandler.openUri(item.type.url)
+                    }
 
                     is AccountItemType.LogOut -> {
                         onLogout()

libs/horizon/src/main/java/com/instructure/horizon/features/account/AccountUiState.kt

@@ -44,7 +44,7 @@ data class AccountItemState(
 
 sealed class AccountItemType(@DrawableRes val icon: Int) {
     data class Open(val route: AccountRoute) : AccountItemType(R.drawable.arrow_forward)
-    data class OpenExternal(val route: AccountRoute) : AccountItemType(R.drawable.open_in_new)
+    data class OpenExternal(val url: String) : AccountItemType(R.drawable.open_in_new)
     data object LogOut : AccountItemType(R.drawable.logout)
     data object SwitchExperience : AccountItemType(R.drawable.swap_horiz)
 }

libs/horizon/src/main/java/com/instructure/horizon/features/account/AccountViewModel.kt

@@ -119,7 +119,7 @@ class AccountViewModel @Inject constructor(
         items = listOf(
             AccountItemState(
                 title = context.getString(R.string.accountReportABug),
-                type = AccountItemType.OpenExternal(AccountRoute.BugReportWebView)
+                type = AccountItemType.OpenExternal("https://community.canvaslms.com/t5/Canvas-Career/Report-an-issue/td-p/662564")
             )
         )
     )

libs/horizon/src/main/java/com/instructure/horizon/features/account/navigation/AccountNavigation.kt

@@ -36,7 +36,6 @@ import com.instructure.horizon.features.account.notifications.AccountNotificatio
 import com.instructure.horizon.features.account.password.AccountPasswordScreen
 import com.instructure.horizon.features.account.profile.AccountProfileScreen
 import com.instructure.horizon.features.account.profile.AccountProfileViewModel
-import com.instructure.horizon.features.account.reportabug.ReportABugWebView
 import com.instructure.horizon.features.home.HomeNavigationRoute
 import com.instructure.horizon.horizonui.animation.enterTransition
 import com.instructure.horizon.horizonui.animation.exitTransition
@@ -91,10 +90,6 @@ fun NavGraphBuilder.accountNavigation(
             val uiState by viewModel.uiState.collectAsState()
             AccountAdvancedScreen(uiState, navController)
         }
-
-        composable(AccountRoute.BugReportWebView.route) {
-            ReportABugWebView(navController)
-        }
     }
 }
 

libs/horizon/src/main/java/com/instructure/horizon/features/account/navigation/AccountRoute.kt

@@ -23,5 +23,4 @@ sealed class AccountRoute(val route: String) {
     data object Notifications : AccountRoute("notifications")
     data object CalendarFeed : AccountRoute("calendar_feed")
     data object Advanced : AccountRoute("advanced")
-    data object BugReportWebView : AccountRoute("report_a_bug")
 }