Merge pull request #126 from harrydbarnes/refactor-onboarding-validation-5103387645092597876

Refactor onboarding validation 5103387645092597876

Author: harrydbarnes

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

@@ -22,10 +22,14 @@ class OnboardingViewModel @Inject constructor(
         _name.value = newName
     }
 
-    fun saveName() {
-        viewModelScope.launch {
-            userPreferencesRepository.saveUserName(name.value)
+    fun saveName(): Boolean {
+        val isNameValid = name.value.isNotBlank()
+        if (isNameValid) {
+            viewModelScope.launch {
+                userPreferencesRepository.saveUserName(name.value)
+            }
         }
+        return isNameValid
     }
 
     fun completeOnboarding() {

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

@@ -98,8 +98,9 @@ fun OnboardingScreen(
                     Button(onClick = {
                         if (currentStep == 0) {
                             // Save Name
-                            if (name.isNotEmpty()) {
-                                viewModel.saveName()
+                            if (!viewModel.saveName()) {
+                                Toast.makeText(context, "Name cannot be blank", Toast.LENGTH_SHORT).show()
+                                return@Button
                             }
                         }