Fixed bug with two Profile screens on first launch

huntj88 · Jul 4, 2021 · 0273abb · 0273abb
1 parent c6565a7
commit 0273abb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/Android/app/src/main/java/me/jameshunt/dhiffiechat/ui/home/HomeScreen.kt b/Android/app/src/main/java/me/jameshunt/dhiffiechat/ui/home/HomeScreen.kt
@@ -137,7 +137,10 @@ fun HomeScreen(
     })
 
     if (!isProfileSetup) {
-        toUserProfile()
+        LaunchedEffect("toUserProfile") {
+            // safely call toUserProfile() once with LaunchedEffect, even though its not a coroutine
+            toUserProfile()
+        }
         return
     }
 

diff --git a/Android/app/src/main/java/me/jameshunt/dhiffiechat/ui/profile/UserProfile.kt b/Android/app/src/main/java/me/jameshunt/dhiffiechat/ui/profile/UserProfile.kt
@@ -1,6 +1,5 @@
 package me.jameshunt.dhiffiechat.ui.profile
 
-import androidx.compose.foundation.isSystemInDarkTheme
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.padding
@@ -12,7 +11,6 @@ import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.saveable.rememberSaveable
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Modifier
-import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.unit.dp
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.ViewModel