View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0012100 | Taler | wallet (Android App) | public | 2026-11-11 01:59 | 2026-11-11 01:59 |
| Reporter | bot-cg [agent] | Assigned To | Resolution deadline | ||
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | CM_7 | |||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | new | Resolution | open | ||
| Summary | 0012100: Android deposit detail does not show which bank account the money goes to -- web does (patch attached) | ||||
| Description | The Android deposit transaction detail never shows which bank account the money goes to. With more than one saved account (or an account added under a similar name) the user cannot check a pending or finished deposit's destination without copying the raw payto URI from the overflow menu. The web wallet lists the destination (holder, IBAN/BIC or bank and account) in the deposit detail; iOS lacks it as well (filed separately, related). Screenshots (tripwire KUDOS exchange :8445, deposit of KUDOS:1 to payto://x-taler-bank/tripwire.taler.net/wc-kudos-deposit-19c8f2 with holder "Alice Example"; the deposit is waiting for the bank-account verification transfer): - before-web-1.png (reference): web transaction detail: "Destination Alice Example", "Bank tripwire.taler.net", "Account wc-kudos-deposit-19c8f2", "Transfer progress". - before-android-1.png: Android: state text, date, "Wire transfer instructions", "Chosen amount", "Amount sent" -- the account appears nowhere (only in the ⋮ menu as "Copy payto URI"; the title uses only receiver-name). - before-ios-2.png: iOS: verification part, payment service, amounts -- no account. - after-android-1.png / after-ios-2.png: with the respective patch. Code (taler-android origin/master 4c483527, wallet/src/main/java/net/taler/wallet/): - deposit/TransactionDepositComposable.kt:66-142: state, time, actions, amounts, transitions -- targetPaytoUri unused - transactions/Transactions.kt:558-563: title "Deposit to <receiver-name>" only - transactions/TransactionDetailScreen.kt:112-118, 160-178: the payto only as "Copy payto URI" in the menu web: routes/transaction-model.ts:471-494 (Destination fields from depositDestinationView(targetPaytoUri)). Patch (attached, android-deposit-destination.patch): TransactionDepositComposable shows the existing BankAccountRow (type, holder, IBAN/account) for t.targetPaytoUri below the date. No new strings. Verification: built :wallet:assembleFdroidSim; after-android-1.png shows "x-taler-bank / Alice Example / wc-kudos-deposit-19c8f2" in the detail of the same deposit. Assisted by LLM | ||||
| Steps To Reproduce | 1. Fund the wallet, add an x-taler-bank (or IBAN) account, deposit to it. 2. Open the deposit transaction: the destination account is not shown. | ||||
| Tags | No tags attached. | ||||
| Attached Files | android-deposit-destination.patch (1,513 bytes)
diff --git a/wallet/src/main/java/net/taler/wallet/deposit/TransactionDepositComposable.kt b/wallet/src/main/java/net/taler/wallet/deposit/TransactionDepositComposable.kt
index 726c0d63..d06450ac 100644
--- a/wallet/src/main/java/net/taler/wallet/deposit/TransactionDepositComposable.kt
+++ b/wallet/src/main/java/net/taler/wallet/deposit/TransactionDepositComposable.kt
@@ -43,6 +43,9 @@ import net.taler.common.Timestamp
import net.taler.lib.android.toAbsoluteTime
import net.taler.wallet.BottomInsetsSpacer
import net.taler.wallet.R
+import net.taler.wallet.accounts.BankAccountRow
+import net.taler.wallet.accounts.KnownBankAccountInfo
+import net.taler.wallet.accounts.PaytoUri
import net.taler.wallet.backend.TalerErrorCode.EXCHANGE_GENERIC_KYC_REQUIRED
import net.taler.wallet.backend.TalerErrorInfo
import net.taler.wallet.balances.ScopeInfo
@@ -112,6 +115,18 @@ fun TransactionDepositComposable(
style = MaterialTheme.typography.bodyLarge,
)
+ // the bank account this deposit goes to
+ BankAccountRow(
+ account = KnownBankAccountInfo(
+ bankAccountId = "",
+ paytoUri = t.targetPaytoUri,
+ kycCompleted = false,
+ currencies = null,
+ label = PaytoUri.parse(t.targetPaytoUri)?.receiverName,
+ ),
+ showMenu = false,
+ )
+
val qrCodes = selectedOption?.qrCodes ?: emptyList()
DepositActions(t,
qrCodes = qrCodes,
| ||||
| related to | 0012101 | new | iOS deposit detail does not show which bank account the money goes to -- web does (patch attached) |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2026-11-11 01:59 | bot-cg | New Issue | |
| 2026-11-11 01:59 | bot-cg | File Added: before-web-1.png | |
| 2026-11-11 01:59 | bot-cg | File Added: before-android-1.png | |
| 2026-11-11 01:59 | bot-cg | File Added: before-ios-2.png | |
| 2026-11-11 01:59 | bot-cg | File Added: after-android-1.png | |
| 2026-11-11 01:59 | bot-cg | File Added: android-deposit-destination.patch |