Merge pull request #173 from hossain-khan/increase-limit-check

[UPDATE] Increase the limit of checker interval

Author: hossain-khan

app/src/main/java/dev/hossain/remotenotify/ui/alertmediumlist/NotificationMediumListScreen.kt

@@ -172,8 +172,8 @@ class NotificationMediumListPresenter
             LaunchedEffect(Unit) {
                 // Set up debounced updates to avoid frequent worker setup
                 workerIntervalFlow
-                    // Wait for 500ms of inactivity
-                    .debounce(500L)
+                    // Wait for 1 second of inactivity
+                    .debounce(1_000L)
                     .collect { minutes ->
                         Timber.d("Worker interval updated: $minutes minutes")
                         appPreferencesDataStore.saveWorkerInterval(minutes)

app/src/main/java/dev/hossain/remotenotify/ui/alertmediumlist/WorkerCheckIntervalConfigUi.kt

@@ -37,6 +37,9 @@ internal fun WorkerConfigCard(
     modifier: Modifier = Modifier,
 ) {
     var intervalSliderValue by remember { mutableFloatStateOf(state.workerIntervalMinutes.toFloat()) }
+    val alertCheckIntervalRangeStart = 60f // 1 hour
+    val alertCheckIntervalRangeEnd = 720f // 12 hours
+
     Card(
         modifier = modifier,
         elevation = CardDefaults.cardElevation(defaultElevation = 2.dp),
@@ -80,8 +83,7 @@ internal fun WorkerConfigCard(
                     intervalSliderValue = intervalValue // Update UI immediately
                     state.eventSink(NotificationMediumListScreen.Event.OnWorkerIntervalUpdated(intervalValue.toLong()))
                 },
-                valueRange = 30f..300f,
-                // steps = 270, // (300-30)/1 to have steps of 1 minute
+                valueRange = alertCheckIntervalRangeStart..alertCheckIntervalRangeEnd,
                 modifier = Modifier.fillMaxWidth(),
                 colors =
                     SliderDefaults.colors(
@@ -98,12 +100,12 @@ internal fun WorkerConfigCard(
                 horizontalArrangement = Arrangement.SpaceBetween,
             ) {
                 Text(
-                    text = "30m",
+                    text = "${(alertCheckIntervalRangeStart / 60).toInt()}h",
                     style = MaterialTheme.typography.bodySmall,
                     color = MaterialTheme.colorScheme.onSurfaceVariant,
                 )
                 Text(
-                    text = "5h",
+                    text = "${(alertCheckIntervalRangeEnd / 60).toInt()}h",
                     style = MaterialTheme.typography.bodySmall,
                     color = MaterialTheme.colorScheme.onSurfaceVariant,
                 )

app/src/main/java/dev/hossain/remotenotify/worker/WorkerRequest.kt

@@ -12,7 +12,7 @@ import androidx.work.WorkManager
 import androidx.work.WorkRequest
 import java.util.concurrent.TimeUnit
 
-internal const val DEFAULT_PERIODIC_INTERVAL_MINUTES = 60L
+internal const val DEFAULT_PERIODIC_INTERVAL_MINUTES = 120L
 internal const val DEVICE_VITALS_CHECKER_DEBUG_WORKER_ID = "onetime-debug-request"
 internal const val DEVICE_VITALS_CHECKER_WORKER_ID = "periodic-health-check"