Merge pull request #124 from harrydbarnes/fix-onboarding-lint-cleanup-14372390311242845095

Fix onboarding lint cleanup 14372390311242845095

Author: harrydbarnes

.gitignore

@@ -13,3 +13,6 @@ local.properties
 *.jks
 *.keystore
 android-sdk/
+
+# Logs
+*.log

app/src/main/AndroidManifest.xml

@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
-<manifest xmlns:android="http://schemas.android.com/apk/res/android">
+<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools">
 
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
@@ -41,6 +42,17 @@
             </intent-filter>
             <meta-data android:name="android.appwidget.provider" android:resource="@xml/widget_info" />
         </receiver>
+
+        <provider
+            android:name="androidx.startup.InitializationProvider"
+            android:authorities="${applicationId}.androidx-startup"
+            android:exported="false"
+            tools:node="merge">
+            <meta-data
+                android:name="androidx.work.WorkManagerInitializer"
+                android:value="androidx.startup"
+                tools:node="remove" />
+        </provider>
     </application>
 
 </manifest>

app/src/main/java/com/example/theloop/OnboardingActivity.kt

@@ -6,7 +6,9 @@ import androidx.activity.ComponentActivity
 import androidx.activity.compose.setContent
 import com.example.theloop.ui.screens.OnboardingScreen
 import com.example.theloop.ui.theme.TheLoopTheme
+import dagger.hilt.android.AndroidEntryPoint
 
+@AndroidEntryPoint
 class OnboardingActivity : ComponentActivity() {
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)

app/src/main/java/com/example/theloop/OnboardingViewModel.kt

@@ -0,0 +1,36 @@
+package com.example.theloop
+
+import androidx.lifecycle.ViewModel
+import androidx.lifecycle.viewModelScope
+import com.example.theloop.data.repository.UserPreferencesRepository
+import dagger.hilt.android.lifecycle.HiltViewModel
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.StateFlow
+import kotlinx.coroutines.flow.asStateFlow
+import kotlinx.coroutines.launch
+import javax.inject.Inject
+
+@HiltViewModel
+class OnboardingViewModel @Inject constructor(
+    private val userPreferencesRepository: UserPreferencesRepository
+) : ViewModel() {
+
+    private val _name = MutableStateFlow("")
+    val name: StateFlow<String> = _name.asStateFlow()
+
+    fun onNameChange(newName: String) {
+        _name.value = newName
+    }
+
+    fun saveName() {
+        viewModelScope.launch {
+            userPreferencesRepository.saveUserName(name.value)
+        }
+    }
+
+    fun completeOnboarding() {
+        viewModelScope.launch {
+            userPreferencesRepository.completeOnboarding()
+        }
+    }
+}

app/src/main/java/com/example/theloop/data/repository/UserPreferencesRepository.kt

@@ -5,6 +5,8 @@ import android.content.SharedPreferences
 import com.example.theloop.R
 import com.example.theloop.utils.AppConstants
 import dagger.hilt.android.qualifiers.ApplicationContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import kotlinx.coroutines.channels.awaitClose
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.callbackFlow
@@ -62,6 +64,14 @@ class UserPreferencesRepository @Inject constructor(
         prefs.edit().putString(AppConstants.KEY_SUMMARY_CACHE, summary).apply()
     }
 
+    suspend fun saveUserName(name: String) = withContext(Dispatchers.IO) {
+        prefs.edit().putString(AppConstants.KEY_USER_NAME, name).commit()
+    }
+
+    suspend fun completeOnboarding() = withContext(Dispatchers.IO) {
+        prefs.edit().putBoolean(AppConstants.KEY_ONBOARDING_COMPLETED, true).commit()
+    }
+
     fun hasLocation(): Boolean {
         return prefs.contains(AppConstants.KEY_LATITUDE) && prefs.contains(AppConstants.KEY_LONGITUDE)
     }

app/src/main/java/com/example/theloop/ui/components/WeatherCard.kt

@@ -9,6 +9,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
+import kotlin.math.roundToInt
 import com.example.theloop.models.WeatherResponse
 import com.example.theloop.utils.AppUtils
 import com.example.theloop.R

app/src/main/java/com/example/theloop/ui/screens/OnboardingScreen.kt

@@ -14,18 +14,20 @@ import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.unit.dp
+import androidx.hilt.navigation.compose.hiltViewModel
 import androidx.health.connect.client.HealthConnectClient
 import androidx.health.connect.client.permission.HealthPermission
 import androidx.health.connect.client.records.StepsRecord
-import com.example.theloop.utils.AppConstants
+import com.example.theloop.OnboardingViewModel
 
 @Composable
 fun OnboardingScreen(
-    onFinish: () -> Unit
+    onFinish: () -> Unit,
+    viewModel: OnboardingViewModel = hiltViewModel()
 ) {
     val context = LocalContext.current
     var currentStep by remember { mutableIntStateOf(0) }
-    var name by remember { mutableStateOf("") }
+    val name by viewModel.name.collectAsState()
     val totalSteps = 5
 
     // Permissions state
@@ -97,21 +99,15 @@ fun OnboardingScreen(
                         if (currentStep == 0) {
                             // Save Name
                             if (name.isNotEmpty()) {
-                                context.getSharedPreferences(AppConstants.PREFS_NAME, Context.MODE_PRIVATE)
-                                    .edit()
-                                    .putString(AppConstants.KEY_USER_NAME, name)
-                                    .apply()
+                                viewModel.saveName()
                             }
                         }
 
                         if (currentStep < totalSteps - 1) {
                             currentStep++
                         } else {
                             // Finish
-                            context.getSharedPreferences(AppConstants.PREFS_NAME, Context.MODE_PRIVATE)
-                                .edit()
-                                .putBoolean(AppConstants.KEY_ONBOARDING_COMPLETED, true)
-                                .apply()
+                            viewModel.completeOnboarding()
                             onFinish()
                         }
                     }) {
@@ -130,7 +126,7 @@ fun OnboardingScreen(
             verticalArrangement = Arrangement.Center
         ) {
             when (currentStep) {
-                0 -> WelcomeStep(name) { name = it }
+                0 -> WelcomeStep(name, viewModel::onNameChange)
                 1 -> PermissionStep(
                     title = "Enable Location",
                     description = "We need your location to show local weather.",