[Issue-3901-v2] Allow signing once for multi-transactions

packages/extension-base/src/background/KoniTypes.ts

@@ -232,13 +232,13 @@
   externalUrl?: string;
   rarity?: string;
   description?: string;
   properties?: Record<any, any> | null;
 }
 
 interface NftItemExtraInfo {
   type?: _AssetType.ERC721 | _AssetType.PSP34 | RMRK_VER; // for sending
   rmrk_ver?: RMRK_VER;
   onChainOption?: any; // for sending PSP-34 tokens, should be done better
   assetHubType?: AssetHubNftType // for sending assetHub nft. There're 2 types nft
 }
 
@@ -543,7 +543,7 @@
   [ExtrinsicType.STAKING_COMPOUNDING]: RequestTuringStakeCompound,
   [ExtrinsicType.STAKING_CANCEL_COMPOUNDING]: RequestTuringCancelStakeCompound,
   [ExtrinsicType.STAKING_CANCEL_UNSTAKE]: RequestStakeCancelWithdrawal,
   [ExtrinsicType.STAKING_POOL_WITHDRAW]: any,
 
   // Yield
   [ExtrinsicType.JOIN_YIELD_POOL]: RequestYieldStepSubmit,
@@ -574,9 +574,9 @@
   [ExtrinsicType.CLAIM_BRIDGE]: RequestClaimBridge
 
   [ExtrinsicType.EVM_EXECUTE]: TransactionConfig,
   [ExtrinsicType.CROWDLOAN]: any,
   [ExtrinsicType.SWAP]: SwapTxData
   [ExtrinsicType.UNKNOWN]: any
 }
 
 export enum ExtrinsicStatus {
@@ -675,7 +675,7 @@
 //     : T extends ExtrinsicType.MINT_VDOT
 //       ? Pick<SubmitBifrostLiquidStaking, 'rewardTokenSlug' | 'estimatedAmountReceived'>
 //       : undefined;
 export interface TransactionHistoryItem<ET extends ExtrinsicType = ExtrinsicType.TRANSFER_BALANCE> {
   origin?: 'app' | 'migration' | 'subsquid' | 'subscan', // 'app' or history source
   callhash?: string,
   signature?: string,
@@ -700,7 +700,7 @@
   tip?: AmountData,
   fee?: AmountData,
   explorerUrl?: string,
   additionalInfo?: any,
   startBlock?: number,
   nonce?: number,
   addressPrefix?: number
@@ -931,12 +931,12 @@
   recipientAddress: string,
 
   nftItemName?: string, // Use for confirmation view only
   params: Record<string, any>,
   nftItem: NftItem
 }
 
 export interface EvmNftTransaction extends ValidateTransactionResponse {
   tx: Record<string, any> | null;
 }
 
 export interface ValidateNetworkResponse {
@@ -1114,6 +1114,7 @@
   requiredPassword?: boolean;
   address?: string;
   networkKey?: string;
+  isPassConfirmation?: boolean;
 }
 
 export interface ConfirmationsQueueItem<T> extends ConfirmationsQueueItemOptions, ConfirmationRequestBase {

packages/extension-base/src/koni/background/handlers/Extension.ts

@@ -1446,8 +1446,7 @@ export default class KoniExtension {
   }
 
   private async makeCrossChainTransfer (inputData: RequestCrossChainTransfer): Promise<SWTransactionResponse> {
-    const { destinationNetworkKey, from, originNetworkKey, to, tokenSlug, transferAll, transferBounceable, value } = inputData;
-
+    const { destinationNetworkKey, from, isPassConfirmation, originNetworkKey, to, tokenSlug, transferAll, transferBounceable, value } = inputData;
     const originTokenInfo = this.#koniState.getAssetBySlug(tokenSlug);
     const destinationTokenInfo = this.#koniState.getXcmEqualAssetByChain(destinationNetworkKey, tokenSlug);
     const [errors, fromKeyPair] = validateXcmTransferRequest(destinationTokenInfo, from, value);
@@ -1552,6 +1551,7 @@ export default class KoniExtension {
       transferNativeAmount: _isNativeToken(originTokenInfo) ? value : '0',
       ignoreWarnings,
       isTransferAll: transferAll,
+      isPassConfirmation,
       errors,
       additionalValidator: additionalValidator,
       eventsHandler: eventsHandler

packages/extension-base/src/services/request-service/handler/EvmRequestHandler.ts

@@ -106,6 +106,10 @@ export default class EvmRequestHandler {
       this.#requestService.popupOpen();
     }
 
+    if (options.isPassConfirmation) {
+      await this.completeConfirmation({ evmSendTransactionRequest: { id, url, isApproved: true, payload: '' } });
+    }
+
     this.#requestService.updateIconV2();
 
     return promise;

packages/extension-base/src/services/transaction-service/index.ts

@@ -869,7 +869,7 @@ export default class TransactionService {
     return ethers.Transaction.from(txObject).unsignedSerialized as HexString;
   }
 
-  private async signAndSendEvmTransaction ({ address, chain, id, transaction, url }: SWTransaction): Promise<TransactionEmitter> {
+  private async signAndSendEvmTransaction ({ address, chain, id, isPassConfirmation, transaction, url }: SWTransaction): Promise<TransactionEmitter> {
     const payload = (transaction as EvmSendTransactionRequest);
     const evmApi = this.state.chainService.getEvmApi(chain);
     const chainInfo = this.state.chainService.getChainInfoByKey(chain);
@@ -1027,7 +1027,7 @@ export default class TransactionService {
           emitter.emit('error', eventData);
         });
     } else {
-      this.state.requestService.addConfirmation(id, url || EXTENSION_REQUEST_URL, 'evmSendTransactionRequest', payload, {})
+      this.state.requestService.addConfirmation(id, url || EXTENSION_REQUEST_URL, 'evmSendTransactionRequest', payload, { isPassConfirmation })
         .then(async ({ isApproved, payload }) => {
           if (isApproved) {
             let signedTransaction: string | undefined;

packages/extension-base/src/services/transaction-service/types.ts

@@ -27,6 +27,7 @@ export interface SWTransaction extends ValidateTransactionResponse, Partial<Pick
   transaction: SubmittableExtrinsic | TransactionConfig | TonTransactionConfig;
   additionalValidator?: (inputTransaction: SWTransactionResponse) => Promise<void>;
   eventsHandler?: (eventEmitter: TransactionEmitter) => void;
+  isPassConfirmation?: boolean
 }
 
 export type SWTransactionResult = Omit<SWTransaction, 'transaction' | 'additionalValidator' | 'eventsHandler'>
@@ -41,6 +42,7 @@ export interface SWTransactionInput extends SwInputBase, Partial<Pick<SWTransact
   errors?: SWTransaction['errors'];
   edAsWarning?: boolean;
   isTransferAll?: boolean;
+  isPassConfirmation?: boolean
   resolveOnDone?: boolean;
   skipFeeValidation?: boolean;
 }

packages/extension-base/src/types/transaction/request.ts

@@ -16,6 +16,7 @@ export interface RequestBaseTransfer {
   value?: string;
   transferAll?: boolean;
   transferBounceable?: boolean;
+  isPassConfirmation?: boolean;
 }
 
 export interface RequestCheckTransfer extends RequestBaseTransfer {

packages/extension-koni-ui/src/Popup/Transaction/variants/SendFund.tsx

@@ -51,6 +51,7 @@ type ComponentProps = {
 interface TransferOptions {
   isTransferAll: boolean;
   isTransferBounceable: boolean;
+  isPassConfirmation: boolean;
 }
 
 function getTokenItems (
@@ -181,6 +182,12 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
 
   const [loading, setLoading] = useState(false);
   const [isTransferAll, setIsTransferAll] = useState(false);
+  const [isSignMultiTransaction, setIsSignMultiTransaction] = useState(true);
+
+  // todo: remove after test
+  useEffect(() => {
+    setIsSignMultiTransaction(false);
+  }, []);
 
   // use this to reinit AddressInput component
   const [addressInputRenderKey, setAddressInputRenderKey] = useState<string>(defaultAddressInputRenderKey);
@@ -458,7 +465,8 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
         to,
         value,
         transferAll: options.isTransferAll,
-        transferBounceable: options.isTransferBounceable
+        transferBounceable: options.isTransferBounceable,
+        isPassConfirmation: options.isPassConfirmation
       });
     }
 
@@ -482,6 +490,8 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
 
   // Submit transaction
   const doSubmit = useCallback((values: TransferParams, options: TransferOptions) => {
+    let isPassConfirmation = false;
+
     if (isShowWarningOnSubmit(values)) {
       return;
     }
@@ -511,12 +521,23 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
           return await submitData(step + 1);
         } else {
           const stepType = processState.steps[step].type;
-          const submitPromise: Promise<SWTransactionResponse> | undefined = stepType === CommonStepType.TOKEN_APPROVAL ? handleBridgeSpendingApproval(values) : handleBasicSubmit(values, options);
+          let submitPromise: Promise<SWTransactionResponse> | undefined;
+
+          if (stepType === CommonStepType.TOKEN_APPROVAL) {
+            submitPromise = handleBridgeSpendingApproval(values);
+          } else {
+            options.isPassConfirmation = isPassConfirmation; // todo: also can set at init of any state except first step
+            submitPromise = handleBasicSubmit(values, options);
+          }
 
           const rs = await submitPromise;
           const success = onSuccess(isLastStep, needRollback)(rs);
 
           if (success) {
+            if (isSignMultiTransaction) {
+              isPassConfirmation = true;
+            }
+
             return await submitData(step + 1);
           } else {
             return false;
@@ -537,7 +558,7 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
           setLoading(false);
         });
     }, 300);
-  }, [handleBasicSubmit, handleBridgeSpendingApproval, isShowWarningOnSubmit, onError, onSuccess, processState]);
+  }, [handleBasicSubmit, isSignMultiTransaction, handleBridgeSpendingApproval, isShowWarningOnSubmit, onError, onSuccess, processState]);
 
   const onSetMaxTransferable = useCallback((value: boolean) => {
     const bnMaxTransfer = new BN(maxTransfer);
@@ -550,7 +571,8 @@ const Component = ({ className = '', isAllAccount, targetAccountProxy }: Compone
   const onSubmit: FormCallbacks<TransferParams>['onFinish'] = useCallback((values: TransferParams) => {
     const options: TransferOptions = {
       isTransferAll: isTransferAll,
-      isTransferBounceable: false
+      isTransferBounceable: false,
+      isPassConfirmation: false
     };
 
     let checkTransferAll = false;