Remove the Context dependency on 'SettingsViewModel' and move to Toas…

…tMessage Composable
android · Mar 21, 2024 · 17bdad1 · 17bdad1
1 parent f9257c9
commit 17bdad1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/app/src/main/java/com/google/android/samples/socialite/ui/home/Settings.kt b/app/src/main/java/com/google/android/samples/socialite/ui/home/Settings.kt
@@ -16,6 +16,7 @@
 
 package com.google.android.samples.socialite.ui.home
 
+import android.widget.Toast
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.fillMaxWidth
@@ -27,6 +28,7 @@ import androidx.compose.material3.Text
 import androidx.compose.material3.TextButton
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
 import androidx.hilt.navigation.compose.hiltViewModel
@@ -38,13 +40,19 @@ fun Settings(
     modifier: Modifier = Modifier,
     viewModel: SettingsViewModel = hiltViewModel(),
 ) {
+    val context = LocalContext.current
+    val clearHistoryMessage = stringResource(R.string.clear_message_history_success)
+
     Column(
         modifier = modifier
             .padding(contentPadding)
             .padding(32.dp),
     ) {
         TextButton(
-            onClick = { viewModel.clearMessages() },
+            onClick = {
+                viewModel.clearMessages()
+                Toast.makeText(context, clearHistoryMessage, Toast.LENGTH_SHORT).show()
+            },
             modifier = Modifier
                 .fillMaxWidth()
                 .heightIn(min = 56.dp),

diff --git a/app/src/main/java/com/google/android/samples/socialite/ui/home/SettingsViewModel.kt b/app/src/main/java/com/google/android/samples/socialite/ui/home/SettingsViewModel.kt
@@ -16,22 +16,18 @@
 
 package com.google.android.samples.socialite.ui.home
 
-import android.content.Context
-import android.widget.Toast
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.google.android.samples.socialite.data.DatabaseManager
 import com.google.android.samples.socialite.repository.ChatRepository
 import dagger.hilt.android.lifecycle.HiltViewModel
-import dagger.hilt.android.qualifiers.ApplicationContext
 import javax.inject.Inject
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
 
 @HiltViewModel
 class SettingsViewModel @Inject constructor(
-    @ApplicationContext private val application: Context,
     private val repository: ChatRepository,
     private val databaseManager: DatabaseManager,
 ) : ViewModel() {
@@ -42,11 +38,6 @@ class SettingsViewModel @Inject constructor(
             withContext(Dispatchers.IO) {
                 databaseManager.wipeAndReinitializeDatabase()
             }
-            Toast.makeText(
-                application.applicationContext,
-                "Messages have been reset",
-                Toast.LENGTH_SHORT,
-            ).show()
         }
     }
 }
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -51,6 +51,7 @@
     <!-- Settings -->
     <eat-comment />
     <string name="clear_message_history">Clear message history</string>
+    <string name="clear_message_history_success">✨ Messages have been reset</string>
 
     <!-- Video Edit Screen -->
     <string name="remove_audio">Remove audio</string>