Refactor settings screen to compose

app/src/main/java/org/groundplatform/android/data/local/LocalValueStore.kt

@@ -115,13 +115,15 @@ constructor(private val preferences: SharedPreferences, private val locale: Loca
       preferences.edit { putString(PrefKeys.MEASUREMENT_UNITS, value) }
     }
 
+  var shouldUploadMediaOverUnmeteredConnectionOnly: Boolean
+    get() = allowThreadDiskReads { preferences.getBoolean(PrefKeys.UPLOAD_MEDIA, false) }
+    set(value) = allowThreadDiskWrites {
+      preferences.edit { putBoolean(PrefKeys.UPLOAD_MEDIA, value) }
+    }
+
   /** Removes all values stored in the local store. */
   fun clear() = allowThreadDiskWrites { preferences.edit { clear() } }
 
-  fun shouldUploadMediaOverUnmeteredConnectionOnly(): Boolean = allowThreadDiskReads {
-    preferences.getBoolean(PrefKeys.UPLOAD_MEDIA, false)
-  }
-
   fun clearLastCameraPosition(surveyId: String) = allowThreadDiskReads {
     preferences.edit { remove(PrefKeys.LAST_VIEWPORT_PREFIX + surveyId) }
   }

app/src/main/java/org/groundplatform/android/data/sync/MediaUploadWorkManager.kt

@@ -27,7 +27,7 @@ class MediaUploadWorkManager
 constructor(private val workManager: WorkManager, private val localValueStore: LocalValueStore) {
 
   private fun preferredNetworkType(): NetworkType =
-    if (localValueStore.shouldUploadMediaOverUnmeteredConnectionOnly()) NetworkType.UNMETERED
+    if (localValueStore.shouldUploadMediaOverUnmeteredConnectionOnly) NetworkType.UNMETERED
     else NetworkType.CONNECTED
 
   /**

app/src/main/java/org/groundplatform/android/di/ViewModelModule.kt

@@ -41,7 +41,6 @@ import org.groundplatform.android.ui.main.MainViewModel
 import org.groundplatform.android.ui.offlineareas.OfflineAreasViewModel
 import org.groundplatform.android.ui.offlineareas.selector.OfflineAreaSelectorViewModel
 import org.groundplatform.android.ui.offlineareas.viewer.OfflineAreaViewerViewModel
-import org.groundplatform.android.ui.settings.SettingsViewModel
 import org.groundplatform.android.ui.syncstatus.SyncStatusViewModel
 import org.groundplatform.android.ui.tos.TermsOfServiceViewModel
 
@@ -143,10 +142,5 @@ abstract class ViewModelModule {
   @ViewModelKey(BaseMapViewModel::class)
   abstract fun bindBaseMapViewModel(viewModel: BaseMapViewModel): ViewModel
 
-  @Binds
-  @IntoMap
-  @ViewModelKey(SettingsViewModel::class)
-  abstract fun bindSettingsViewModel(viewModel: SettingsViewModel): ViewModel
-
   @Binds abstract fun bindViewModelFactory(factory: ViewModelFactory): ViewModelProvider.Factory
 }

app/src/main/java/org/groundplatform/android/repository/UserRepository.kt

@@ -128,7 +128,15 @@ constructor(
       UserSettings(
         language = selectedLanguage,
         measurementUnits = MeasurementUnits.valueOf(selectedLengthUnit),
-        shouldUploadPhotosOnWifiOnly = shouldUploadMediaOverUnmeteredConnectionOnly(),
+        shouldUploadPhotosOnWifiOnly = shouldUploadMediaOverUnmeteredConnectionOnly,
       )
     }
+
+  fun setUserSettings(userSettings: UserSettings) {
+    with(localValueStore) {
+      selectedLanguage = userSettings.language
+      selectedLengthUnit = userSettings.measurementUnits.name
+      shouldUploadMediaOverUnmeteredConnectionOnly = userSettings.shouldUploadPhotosOnWifiOnly
+    }
+  }
 }

app/src/main/java/org/groundplatform/android/ui/settings/SettingsActivity.kt

@@ -15,22 +15,45 @@
  */
 package org.groundplatform.android.ui.settings
 
+import android.content.Intent
+import android.net.Uri
 import android.os.Bundle
+import androidx.activity.compose.setContent
+import androidx.appcompat.app.AppCompatDelegate
+import androidx.core.os.LocaleListCompat
 import dagger.hilt.android.AndroidEntryPoint
-import org.groundplatform.android.databinding.SettingsActivityBinding
 import org.groundplatform.android.ui.common.AbstractActivity
+import org.groundplatform.android.ui.main.MainActivity
+import org.groundplatform.android.ui.theme.AppTheme
 
 @AndroidEntryPoint
 class SettingsActivity : AbstractActivity() {
   override fun onCreate(savedInstanceState: Bundle?) {
     super.onCreate(savedInstanceState)
+    setContent {
+      AppTheme {
+        SettingsScreen(
+          onBack = { finish() },
+          onLocaleChanged = { locale -> applyLocaleAndRestart(locale) },
+          onVisitWebsiteClick = { uri: Uri -> openWebsite(uri) },
+        )
+      }
+    }
+  }
 
-    val binding = SettingsActivityBinding.inflate(layoutInflater)
-    setContentView(binding.root)
+  private fun applyLocaleAndRestart(languageCode: String) {
+    val appLocale = LocaleListCompat.forLanguageTags(languageCode)
+    AppCompatDelegate.setApplicationLocales(appLocale)
 
-    with(binding.settingsToolbar) {
-      setSupportActionBar(this)
-      setNavigationOnClickListener { finish() }
-    }
+    val intent =
+      Intent(this, MainActivity::class.java).apply {
+        flags = Intent.FLAG_ACTIVITY_NEW_TASK or Intent.FLAG_ACTIVITY_CLEAR_TASK
+      }
+    startActivity(intent)
+  }
+
+  private fun openWebsite(uri: Uri) {
+    val intent = Intent(Intent.ACTION_VIEW, uri)
+    startActivity(intent)
   }
 }