#1440 fix: do not show button not detected bottom sheet repeatedly

sds100 · sds100 · commit 1b32b70cdf2f · 2025-02-17T16:52:46.000-06:00
diff --git a/app/src/main/java/io/github/sds100/keymapper/mappings/keymaps/trigger/BaseConfigTriggerViewModel.kt b/app/src/main/java/io/github/sds100/keymapper/mappings/keymaps/trigger/BaseConfigTriggerViewModel.kt
@@ -273,7 +273,10 @@ abstract class BaseConfigTriggerViewModel(
                 }
         }
 
-        recordTrigger.state.onEach { state ->
+        // Drop the first state in case it is in the Completed state so the
+        // "button not detected" bottom sheet isn't shown when
+        // the screen is opened.
+        recordTrigger.state.drop(1).onEach { state ->
             if (state is RecordTriggerState.Completed &&
                 state.recordedKeys.isEmpty() &&
                 onboarding.showNoKeysDetectedBottomSheet.first() &&

Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,10 @@ abstract class BaseConfigTriggerViewModel(`
`273`	`273`	`}`
`274`	`274`	`}`
`275`	`275`
`276`		`- recordTrigger.state.onEach { state ->`
	`276`	`+ // Drop the first state in case it is in the Completed state so the`
	`277`	`+ // "button not detected" bottom sheet isn't shown when`
	`278`	`+ // the screen is opened.`
	`279`	`+ recordTrigger.state.drop(1).onEach { state ->`
`277`	`280`	`if (state is RecordTriggerState.Completed &&`
`278`	`281`	`state.recordedKeys.isEmpty() &&`
`279`	`282`	`onboarding.showNoKeysDetectedBottomSheet.first() &&`