fix: keep fixed paykit requests on confirmation with an asset switch

app/src/main/java/to/bitkit/ui/screens/wallets/send/SendConfirmScreen.kt

@@ -592,10 +592,10 @@ private fun OnChainDetails(
                         stringResource(R.string.wallet__savings__title)
                     },
                     color = if (uiState.hardwareWalletId != null) Colors.Blue else Colors.Brand,
-                    enabled = uiState.canSwitchBitcoinFundingSource && interactionsEnabled,
+                    enabled = uiState.canSwitchConfirmFundingSource && interactionsEnabled,
                     isLoading = uiState.isFundingSourceLoading,
                     clickable = interactionsEnabled && !uiState.isRetainedHardwarePayment,
-                    icon = R.drawable.ic_transfer.takeIf { uiState.canSwitchBitcoinFundingSource },
+                    icon = R.drawable.ic_transfer.takeIf { uiState.canSwitchConfirmFundingSource },
                     onClick = { onEvent(SendEvent.PaymentMethodSwitch) },
                     modifier = Modifier.testTag("SendConfirmAssetButton")
                 )
@@ -744,9 +744,9 @@ private fun LightningDetails(
                 NumberPadActionButton(
                     text = stringResource(R.string.wallet__spending__title),
                     color = Colors.Purple,
-                    enabled = uiState.canSwitchBitcoinFundingSource && interactionsEnabled,
+                    enabled = uiState.canSwitchConfirmFundingSource && interactionsEnabled,
                     isLoading = uiState.isFundingSourceLoading,
-                    icon = R.drawable.ic_transfer.takeIf { uiState.canSwitchBitcoinFundingSource },
+                    icon = R.drawable.ic_transfer.takeIf { uiState.canSwitchConfirmFundingSource },
                     onClick = { onEvent(SendEvent.PaymentMethodSwitch) },
                     modifier = Modifier.testTag("SendConfirmAssetButton")
                 )

app/src/main/java/to/bitkit/ui/screens/wallets/usdt/PaykitUsdtReview.kt

@@ -32,6 +32,7 @@ import to.bitkit.ui.screens.wallets.send.AutoPayOverlay
 import to.bitkit.ui.shared.modifiers.sheetHeight
 import to.bitkit.ui.theme.Colors
 import to.bitkit.viewmodels.AppViewModel
+import to.bitkit.viewmodels.SendEvent
 import to.bitkit.viewmodels.SendUiState
 import kotlin.time.Duration.Companion.seconds
 
@@ -83,6 +84,11 @@ fun PaykitUsdtReview(
     LaunchedEffect(Unit) {
         preparePayment(viewModel, app, uiState) { automaticPayment = it }
     }
+    LaunchedEffect(state.quote, state.error) {
+        if (state.quote == null && state.error == R.string.usdt__error_expired && uiState.isPaymentRequest) {
+            viewModel.requote(requireNotNull(uiState.usdtRecipient), uiState.paykitAmount, UsdtDestination.ARBITRUM)
+        }
+    }
     LaunchedEffect(state.submitted, lifecycle) {
         lifecycle.repeatOnLifecycle(Lifecycle.State.STARTED) {
             reconcileSubmittedPayment(viewModel, app)
@@ -96,7 +102,7 @@ fun PaykitUsdtReview(
             confirmText = stringResource(R.string.wallet__send_yes),
             dismissText = stringResource(R.string.common__cancel),
             onConfirm = viewModel::acceptWarning,
-            onDismiss = viewModel::edit
+            onDismiss = viewModel::dismissWarning
         )
     }
     val transfer = wallet.transfers.firstOrNull { it.id == state.quote?.id }
@@ -151,6 +157,8 @@ fun PaykitUsdtReview(
                         paymentRequestNote = uiState.paymentRequestNote,
                         isMerchantRequest = uiState.isMerchantPaymentRequest,
                         contact = uiState.contactPaymentProfile,
+                        onSwitchAsset = { app.setSendEvent(SendEvent.PaymentMethodSwitch) }
+                            .takeIf { uiState.canSwitchConfirmFundingSource },
                     )
                 }
             }

app/src/main/java/to/bitkit/ui/screens/wallets/usdt/UsdtSendContent.kt

@@ -11,6 +11,7 @@ import androidx.compose.foundation.layout.ColumnScope
 import androidx.compose.foundation.layout.FlowRow
 import androidx.compose.foundation.layout.IntrinsicSize
 import androidx.compose.foundation.layout.Row
+import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxHeight
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
@@ -127,6 +128,7 @@ internal fun UsdtPaymentContent(
     contact: PubkyProfile? = null,
     onDestinationChange: ((UsdtDestination) -> Unit)? = null,
     destinations: ImmutableList<UsdtDestination> = Env.usdtDestinations.toImmutableList(),
+    onSwitchAsset: (() -> Unit)? = null,
 ) {
     if (page == UsdtPage.WALLET) return
 
@@ -161,6 +163,9 @@ internal fun UsdtPaymentContent(
         }
     }
     val showSubmitted = pending || (completed && hasShownSubmitted)
+    // A payment request's amount is fixed, so its review waits for the quote on a confirm screen, not on Amount.
+    val preparingConfirmation = quote == null && page == UsdtPage.AMOUNT && isPaymentRequest && !amountEditable &&
+        state.error == null
     if (showSuccess && completed && quote != null) {
         NewTransactionSheetView(
             details = NewTransactionSheetDetails(
@@ -188,7 +193,7 @@ internal fun UsdtPaymentContent(
         showSubmitted && failed -> R.string.wallet__send_error_tx_failed
         showSubmitted && submissionError != null -> R.string.common__error
         showSubmitted -> R.string.usdt__submitted
-        quote != null && isPaymentRequest -> R.string.wallet__payment_request
+        (quote != null || preparingConfirmation) && isPaymentRequest -> R.string.wallet__payment_request
         quote != null -> R.string.wallet__send_review
         page == UsdtPage.AMOUNT -> R.string.usdt__amount
         else -> R.string.usdt__send_title
@@ -222,7 +227,9 @@ internal fun UsdtPaymentContent(
                 onConfirm,
                 completed,
                 onEditAmount = { if (amountEditable) onPageChange(UsdtPage.AMOUNT) },
+                onSwitchAsset = onSwitchAsset,
             )
+            preparingConfirmation -> UsdtConfirmationLoading(amount, contact, paymentRequestNote)
             page == UsdtPage.AMOUNT -> UsdtAmountContent(
                 amount,
                 destination,
@@ -560,6 +567,7 @@ private fun UsdtConfirmation(
     onConfirm: () -> Unit,
     completed: Boolean,
     onEditAmount: () -> Unit,
+    onSwitchAsset: (() -> Unit)?,
 ) {
     var showDetails by rememberSaveable(quote.id) { mutableStateOf(false) }
     val swipeProgress = remember { mutableFloatStateOf(0f) }
@@ -579,6 +587,7 @@ private fun UsdtConfirmation(
                     isPaymentRequest,
                     isMerchantRequest,
                     showDetails,
+                    onSwitchAsset = onSwitchAsset?.takeUnless { state.busy || completed },
                 ) {
                     if (!state.busy && !completed) onEditAmount()
                 }
@@ -619,6 +628,31 @@ private fun UsdtConfirmation(
     }
 }
 
+@Composable
+private fun UsdtConfirmationLoading(amount: String, contact: PubkyProfile?, note: String?) {
+    Column(modifier = Modifier.fillMaxSize().testTag("UsdtConfirmLoading")) {
+        UsdtAmountHeader(amount.ifEmpty { "0" })
+        VerticalSpacer(24.dp)
+        PaymentRequestSummary(
+            profile = contact,
+            note = note?.trim()?.takeIf { it.isNotEmpty() },
+            accentColor = Colors.Usdt,
+        )
+        Spacer(modifier = Modifier.weight(1f))
+        SwipeToConfirm(
+            text = stringResource(R.string.wallet__send_swipe),
+            color = Colors.Usdt,
+            onConfirm = {},
+            enabled = false,
+            loading = true,
+            confirmed = false,
+            progress = remember { mutableFloatStateOf(0f) },
+            textStyle = AppTextStyles.BodySSB.copy(letterSpacing = 0.sp),
+            modifier = Modifier.fillMaxWidth(),
+        )
+    }
+}
+
 @Composable
 private fun ColumnScope.UsdtReviewDetailsButton(showDetails: Boolean, onClick: () -> Unit) {
     PrimaryButton(
@@ -667,6 +701,7 @@ private fun UsdtReviewHeader(
     isMerchantRequest: Boolean,
     showDetails: Boolean,
     modifier: Modifier = Modifier,
+    onSwitchAsset: (() -> Unit)? = null,
     onEditAmount: () -> Unit,
 ) {
     Column(modifier = modifier) {
@@ -677,7 +712,7 @@ private fun UsdtReviewHeader(
         )
         VerticalSpacer(24.dp)
         if (showDetails) {
-            UsdtQuoteDetails(quote, contact, isMerchantRequest)
+            UsdtQuoteDetails(quote, contact, isMerchantRequest, onSwitchAsset)
             note?.let {
                 VerticalSpacer(16.dp)
                 PaymentRequestInvoiceNote(it)
@@ -690,16 +725,23 @@ private fun UsdtReviewHeader(
 }
 
 @Composable
-private fun UsdtQuoteDetails(quote: UsdtQuote, contact: PubkyProfile?, isMerchantRequest: Boolean) {
+private fun UsdtQuoteDetails(
+    quote: UsdtQuote,
+    contact: PubkyProfile?,
+    isMerchantRequest: Boolean,
+    onSwitchAsset: (() -> Unit)?,
+) {
     var showFullAddress by rememberSaveable(quote.recipient) { mutableStateOf(false) }
     Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) {
         UsdtDetailField(caption = stringResource(R.string.usdt__pay_from), modifier = Modifier.weight(1f)) {
             NumberPadActionButton(
                 text = "USDT",
                 color = Colors.Usdt,
-                enabled = false,
+                enabled = onSwitchAsset != null,
+                icon = R.drawable.ic_transfer.takeIf { onSwitchAsset != null },
                 textStyle = AppTextStyles.CaptionM.copy(letterSpacing = 1.sp),
-                onClick = {},
+                onClick = { onSwitchAsset?.invoke() },
+                modifier = Modifier.testTag("UsdtPayFrom"),
             )
         }
         UsdtDetailCell(

app/src/main/java/to/bitkit/ui/screens/wallets/usdt/UsdtViewModel.kt

@@ -258,6 +258,21 @@ class UsdtViewModel @Inject constructor(
     fun cancelAuthentication() = _state.update { it.copy(authenticationRequired = false) }
     fun edit() = _state.update { UsdtSendState(refreshError = it.refreshError) }
 
+    /** Keeps the reviewed quote when a warning is declined; the warning shows again on the next swipe. */
+    fun dismissWarning() = _state.update { it.copy(warning = null) }
+
+    // One automatic requote per expired submission; a later expiry waits for the user's own retry.
+    private var requotedExpiredSubmission = false
+
+    /** Replaces an expired quote and keeps the review, telling the user to check the new fee and swipe again. */
+    fun requote(recipient: String, amount: String, destination: UsdtDestination) = viewModelScope.launch {
+        if (_state.value.error != R.string.usdt__error_expired || requotedExpiredSubmission) return@launch
+        requotedExpiredSubmission = true
+        _state.update { it.copy(error = null) }
+        quote(recipient, amount, destination).join()
+        _state.update { if (it.quote != null) it.copy(error = R.string.usdt__error_quote_refreshed) else it }
+    }
+
     fun retryRecovery() = viewModelScope.launch {
         val current = state.value
         val quote = current.quote ?: return@launch
@@ -290,6 +305,7 @@ class UsdtViewModel @Inject constructor(
         if (_state.value.busy) return@launch
         maximumQuote = null
         preparedQuote = null
+        requotedExpiredSubmission = false
         _state.update { it.copy(busy = true, error = null) }
         runSuspendCatching {
             val payment = sendPayment

app/src/main/java/to/bitkit/ui/sheets/SendSheet.kt

@@ -120,8 +120,9 @@ internal fun NavController.navigateToCoinSelection() {
 internal fun NavController.navigateToSendAmount(startDestination: SendRoute) {
     if (popBackStack(SendRoute.Amount, inclusive = false)) return
     navigateTo(SendRoute.Amount) {
-        if (startDestination == SendRoute.Confirm) {
-            popUpTo(SendRoute.Confirm) { inclusive = true }
+        if (startDestination == SendRoute.Confirm || startDestination == SendRoute.UsdtReview) {
+            // A Pay From switch may have replaced the starting confirmation, so clear whichever one is there.
+            popUpTo(graph.id)
         }
     }
 }
@@ -283,6 +284,14 @@ fun SendSheet(
                         is SendEffect.NavigateToCoinSelection -> navController.navigateToCoinSelection()
                         is SendEffect.NavigateToConfirm -> navController.navigateTo(SendRoute.Confirm)
                         is SendEffect.NavigateToUsdtReview -> navController.navigateTo(SendRoute.UsdtReview)
+                        is SendEffect.ShowPaykitConfirmation -> {
+                            // Replace the other asset's confirmation, so Back never returns to its stale terms.
+                            val replaced = when (it.route) {
+                                SendRoute.Confirm -> SendRoute.UsdtReview
+                                else -> SendRoute.Confirm
+                            }
+                            navController.navigateTo(it.route) { popUpTo(replaced) { inclusive = true } }
+                        }
                         is SendEffect.NavigateToHardwareSign -> navController.navigateTo(SendRoute.HardwareSign)
                         is SendEffect.PopBack -> navController.popBackStack(it.route, inclusive = false)
                         is SendEffect.PaymentSuccess -> {