From 15525e65128c197558acae8d0de0818f3736c3f2 Mon Sep 17 00:00:00 2001
From: jvsena42 <jvsena16@gmail.com>
Date: Tue, 30 Dec 2025 09:16:54 -0300
Subject: [PATCH] fix: always clean queue on sheet dismiss

---
 .../main/java/to/bitkit/ui/nav/entries/SheetEntries.kt   | 6 +++---
 .../to/bitkit/utils/timedsheets/TimedSheetManager.kt     | 9 ++-------
 app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt   | 3 +--
 .../to/bitkit/utils/timedsheets/TimedSheetManagerTest.kt | 4 ++--
 4 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/app/src/main/java/to/bitkit/ui/nav/entries/SheetEntries.kt b/app/src/main/java/to/bitkit/ui/nav/entries/SheetEntries.kt
index 6de8e12ce..a20a389b0 100644
--- a/app/src/main/java/to/bitkit/ui/nav/entries/SheetEntries.kt
+++ b/app/src/main/java/to/bitkit/ui/nav/entries/SheetEntries.kt
@@ -745,7 +745,7 @@ private fun EntryProviderScope<NavKey>.sheetFlowEntries(
         }
         BackgroundPaymentsIntroSheet(
             onContinue = {
-                appViewModel.dismissTimedSheet(skipQueue = true)
+                appViewModel.dismissTimedSheet()
                 navigator.navigate(Routes.BackgroundPayments.Settings)
             },
         )
@@ -759,7 +759,7 @@ private fun EntryProviderScope<NavKey>.sheetFlowEntries(
         }
         QuickPayIntroSheet(
             onContinue = {
-                appViewModel.dismissTimedSheet(skipQueue = true)
+                appViewModel.dismissTimedSheet()
                 navigator.navigate(Routes.QuickPay.Settings)
             },
         )
@@ -777,7 +777,7 @@ private fun EntryProviderScope<NavKey>.sheetFlowEntries(
             learnMoreClick = {
                 val intent = Intent(Intent.ACTION_VIEW, Env.STORING_BITCOINS_URL.toUri())
                 context.startActivity(intent)
-                appViewModel.dismissTimedSheet(skipQueue = true)
+                appViewModel.dismissTimedSheet()
                 navigator.goBack()
             },
         )
diff --git a/app/src/main/java/to/bitkit/utils/timedsheets/TimedSheetManager.kt b/app/src/main/java/to/bitkit/utils/timedsheets/TimedSheetManager.kt
index 3260e5e1b..71f34852e 100644
--- a/app/src/main/java/to/bitkit/utils/timedsheets/TimedSheetManager.kt
+++ b/app/src/main/java/to/bitkit/utils/timedsheets/TimedSheetManager.kt
@@ -42,7 +42,7 @@ class TimedSheetManager(private val scope: CoroutineScope) {
         checkJob = null
     }
 
-    fun dismissCurrentSheet(skipQueue: Boolean = false) {
+    fun dismissCurrentSheet() {
         if (currentTimedSheet == null) return
 
         scope.launch {
@@ -50,12 +50,7 @@ class TimedSheetManager(private val scope: CoroutineScope) {
             _currentSheet.value = null
             currentTimedSheet = null
 
-            if (skipQueue) {
-                Logger.debug("Clearing timed sheet queue", context = TAG)
-            } else {
-                delay(CHECK_DELAY_MILLIS)
-                checkAndShowNextSheet()
-            }
+            Logger.debug("Clearing timed sheet queue", context = TAG)
         }
     }
 
diff --git a/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt b/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt
index d3b67c0aa..196ee19bd 100644
--- a/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt
+++ b/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt
@@ -1736,8 +1736,7 @@ class AppViewModel @Inject constructor(
 
     fun onLeftHome() = timedSheetManager.onHomeScreenExited()
 
-    fun dismissTimedSheet(skipQueue: Boolean = false) =
-        timedSheetManager.dismissCurrentSheet(skipQueue)
+    fun dismissTimedSheet() = timedSheetManager.dismissCurrentSheet()
 
     private suspend fun checkCriticalAppUpdate() = withContext(bgDispatcher) {
         delay(SCREEN_TRANSITION_DELAY_MS)
diff --git a/app/src/test/java/to/bitkit/utils/timedsheets/TimedSheetManagerTest.kt b/app/src/test/java/to/bitkit/utils/timedsheets/TimedSheetManagerTest.kt
index 2e698039f..24228de27 100644
--- a/app/src/test/java/to/bitkit/utils/timedsheets/TimedSheetManagerTest.kt
+++ b/app/src/test/java/to/bitkit/utils/timedsheets/TimedSheetManagerTest.kt
@@ -103,7 +103,7 @@ class TimedSheetManagerTest : BaseUnitTest() {
 
         assertEquals(TimedSheetType.APP_UPDATE, sut.currentSheet.value)
 
-        sut.dismissCurrentSheet(skipQueue = true)
+        sut.dismissCurrentSheet()
         testScope.advanceTimeBy(100)
 
         assertNull(sut.currentSheet.value)
@@ -139,7 +139,7 @@ class TimedSheetManagerTest : BaseUnitTest() {
 
         assertEquals(TimedSheetType.BACKUP, sut.currentSheet.value)
 
-        sut.dismissCurrentSheet(skipQueue = false)
+        sut.dismissCurrentSheet()
         testScope.advanceTimeBy(100)
 
         assertNull(sut.currentSheet.value)