From f1bf7705a7ec8cf90d0defa1234a3cce40a339d5 Mon Sep 17 00:00:00 2001 From: Balazs Gerlei Date: Fri, 8 Nov 2024 13:12:18 +0100 Subject: [PATCH] NEVISACCESSAPP-6287: Fix comment for TransactionConfirmationResponse and require the account to exist - Now that transaction confirmation is after Account Selection, it should always be determined at this point --- .../model/response/TransactionConfirmationResponse.kt | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/ch/nevis/exampleapp/coroutines/domain/model/response/TransactionConfirmationResponse.kt b/app/src/main/java/ch/nevis/exampleapp/coroutines/domain/model/response/TransactionConfirmationResponse.kt index 96a02b9..feda529 100644 --- a/app/src/main/java/ch/nevis/exampleapp/coroutines/domain/model/response/TransactionConfirmationResponse.kt +++ b/app/src/main/java/ch/nevis/exampleapp/coroutines/domain/model/response/TransactionConfirmationResponse.kt @@ -10,9 +10,8 @@ import ch.nevis.mobile.sdk.api.localdata.Account /** * [Response] class that indicates a transaction confirmation has to be started. - * Typically the received [Account] set is shown to the user and he/she selects one of them. - * After the account selection [ch.nevis.exampleapp.coroutines.domain.usecase.SelectAccountUseCase] is called - * to continue the operation. + * After the transaction confirmation, [ch.nevis.exampleapp.coroutines.domain.usecase.SelectAccountUseCase] + * should be called with the contained account to continue the operation. * * @constructor Creates a new instance. * @param account The previously selected account. @@ -24,7 +23,7 @@ class TransactionConfirmationResponse ( /** * The previously selected account. */ - val account: Account? = null, + val account: Account, /** * The optional transaction data/message that is sent during an authentication process.