Ensure correct args parsing

Author: hernan-clich

src/utils/nftTransfer.ts

@@ -24,18 +24,54 @@ export const transferNFT = async ({
   fromChainName,
 }: TransferNFTParams): Promise<ethers.ContractTransactionResponse> => {
   try {
+    // Validate input parameters
+    if (
+      !tokenId ||
+      !currentChainContractAddress ||
+      !destinationChain ||
+      !receiverAddress
+    ) {
+      throw new Error('Missing required parameters for NFT transfer');
+    }
+
     // Get the ZRC20 token address for the destination chain
     const destinationAddress = getZRC20Address(destinationChain);
     if (!destinationAddress) {
-      throw new Error(`ZRC20 address not found for destination chain: ${destinationChain}`);
+      throw new Error(
+        `ZRC20 address not found for destination chain: ${destinationChain}`
+      );
     }
 
+    console.log('Transfer parameters:', {
+      tokenId,
+      currentChainContractAddress,
+      destinationChain,
+      destinationAddress,
+      receiverAddress,
+      fromChainName,
+    });
+
     // Create the NFT contract instance using the current chain's contract address
     const nftContract = createNFTContract(currentChainContractAddress, signer);
 
-    // Step 1: Approve the contract to transfer the NFT
+    // Verify ownership before proceeding
+    const userAddress = await signer.getAddress();
+    console.log('Verifying NFT ownership...', { tokenId, userAddress });
+
+    // Check if the user owns the NFT (this is a basic check, the contract will also verify)
+    try {
+      const balance = await nftContract.balanceOf(userAddress);
+      if (balance === 0n) {
+        throw new Error('You do not own any NFTs from this contract');
+      }
+    } catch (error) {
+      console.warn('Could not verify NFT balance:', error);
+      // Continue anyway as the contract will handle the ownership check
+    }
+
+    // Step 1: Approve the ZRC20 contract to transfer the NFT
     console.log('Approving NFT for transfer...');
-    const approveTx = await nftContract.approve(currentChainContractAddress, tokenId, {
+    const approveTx = await nftContract.approve(destinationAddress, tokenId, {
       gasLimit: 1000000,
     });
     await approveTx.wait();
@@ -47,19 +83,19 @@ export const transferNFT = async ({
       receiverAddress,
       destinationAddress,
     });
-    const result = await nftContract.transferCrossChain(
+    const result = (await nftContract.transferCrossChain(
       tokenId,
       receiverAddress,
       destinationAddress,
       {
         gasLimit: 1000000,
         value: ethers.parseEther('0.01'), // Start with 0 gas amount, can be adjusted
       }
-    ) as ethers.ContractTransactionResponse;
+    )) as ethers.ContractTransactionResponse;
 
     // Wait for the transaction to be mined
     const receipt = await result.wait();
-    
+
     if (!receipt) {
       throw new Error('Transaction receipt not found');
     }
@@ -69,7 +105,10 @@ export const transferNFT = async ({
     // Update NFT storage with transfer information
     try {
       updateNFTTransfer(tokenId, contractAddress, {
-        fromChainId: await signer.provider?.getNetwork().then(network => network.chainId.toString()) || '0',
+        fromChainId:
+          (await signer.provider
+            ?.getNetwork()
+            .then((network) => network.chainId.toString())) || '0',
         fromChainName,
         toChainId: getChainId(destinationChain),
         toChainName: destinationChain,