fix batching claim conditions

Author: MananTank

apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/_common/form.ts

@@ -49,8 +49,11 @@ export type CreateNFTCollectionFunctions = {
       contractAddress: string;
     }>;
     setClaimConditions: (params: {
-      nftCollectionInfo: NFTCollectionInfoFormValues;
-      nftBatch: NFTMetadataWithPrice[];
+      values: CreateNFTCollectionAllValues;
+      batch: {
+        startIndex: number;
+        endIndex: number;
+      };
     }) => Promise<void>;
     lazyMintNFTs: (values: CreateNFTCollectionAllValues) => Promise<void>;
   };

apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/create-nft-page.tsx

@@ -24,16 +24,12 @@ import { useActiveAccount } from "thirdweb/react";
 import { maxUint256 } from "thirdweb/utils";
 import { parseError } from "utils/errorParser";
 import { useAddContractToProject } from "../../../hooks/project-contracts";
-import type {
-  CreateNFTCollectionAllValues,
-  NFTCollectionInfoFormValues,
-} from "./_common/form";
+import type { CreateNFTCollectionAllValues } from "./_common/form";
 import {
   getNFTDeploymentTrackingData,
   getNFTStepTrackingData,
 } from "./_common/tracking";
 import { CreateNFTPageUI } from "./create-nft-page-ui";
-import type { NFTMetadataWithPrice } from "./upload-nfts/batch-upload/process-files";
 
 export function CreateNFTPage(props: {
   accountAddress: string;
@@ -336,34 +332,37 @@ export function CreateNFTPage(props: {
   }
 
   async function handleSetClaimConditionsERC1155(params: {
-    nftCollectionInfo: NFTCollectionInfoFormValues;
-    nftBatch: NFTMetadataWithPrice[];
+    values: CreateNFTCollectionAllValues;
+    batch: {
+      startIndex: number;
+      endIndex: number;
+    };
   }) {
-    const { nftCollectionInfo, nftBatch } = params;
+    const { values, batch } = params;
     const { contract, activeAccount } = getContractAndAccount({
-      chain: nftCollectionInfo.chain,
+      chain: values.collectionInfo.chain,
     });
 
-    const nfts = nftBatch;
+    const nfts = values.nfts.slice(batch.startIndex, batch.endIndex);
 
     // fetch nfts
     const fetchedNFTs = await getNFTs1155({
       contract,
-      start: 0,
+      start: batch.startIndex,
+      count: batch.endIndex - batch.startIndex,
     });
 
-    const transactions = nfts.map((uploadedNFT, nftIndex) => {
-      const fetchedNFT =
-        fetchedNFTs.find(
-          (nft) =>
-            nft.metadata.name === uploadedNFT.name &&
-            nft.metadata.description === uploadedNFT.description,
-        ) || fetchedNFTs[nftIndex];
+    const transactions = nfts.map((uploadedNFT, i) => {
+      const fetchedNFT = fetchedNFTs[i];
 
       if (!fetchedNFT) {
         throw new Error("Failed to find NFT");
       }
 
+      if (fetchedNFT.metadata.name !== uploadedNFT.name) {
+        throw new Error("Failed to find NFT in batch");
+      }
+
       return setClaimConditions1155({
         contract,
         tokenId: fetchedNFT.id,
@@ -395,7 +394,7 @@ export function CreateNFTPage(props: {
       getNFTStepTrackingData({
         action: "claim-conditions",
         ercType: "erc1155",
-        chainId: Number(nftCollectionInfo.chain),
+        chainId: Number(values.collectionInfo.chain),
         status: "attempt",
       }),
     );
@@ -410,7 +409,7 @@ export function CreateNFTPage(props: {
         getNFTStepTrackingData({
           action: "claim-conditions",
           ercType: "erc1155",
-          chainId: Number(nftCollectionInfo.chain),
+          chainId: Number(values.collectionInfo.chain),
           status: "success",
         }),
       );
@@ -423,7 +422,7 @@ export function CreateNFTPage(props: {
         getNFTStepTrackingData({
           action: "claim-conditions",
           ercType: "erc1155",
-          chainId: Number(nftCollectionInfo.chain),
+          chainId: Number(values.collectionInfo.chain),
           status: "error",
           errorMessage,
         }),
@@ -456,8 +455,8 @@ export function CreateNFTPage(props: {
               ercType: "erc1155",
             });
           },
-          setClaimConditions: async (formValues) => {
-            return handleSetClaimConditionsERC1155(formValues);
+          setClaimConditions: async (params) => {
+            return handleSetClaimConditionsERC1155(params);
           },
         },
         erc721: {

apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/assets/create/nft/launch/launch-nft.tsx

@@ -164,7 +164,6 @@ export function LaunchNFT(props: {
       if (ercType === "erc721") {
         await props.createNFTFunctions.erc721.setClaimConditions(formValues);
       } else {
-        const { nfts, collectionInfo } = formValues;
         if (batchCount > 1) {
           const batchStartIndex = batchesProcessedRef.current;
           for (
@@ -184,19 +183,22 @@ export function LaunchNFT(props: {
             }
 
             await props.createNFTFunctions.erc1155.setClaimConditions({
-              nftCollectionInfo: collectionInfo,
-              nftBatch: nfts.slice(
-                batchIndex * batchSize,
-                (batchIndex + 1) * batchSize,
-              ),
+              values: formValues,
+              batch: {
+                startIndex: batchIndex * batchSize,
+                endIndex: (batchIndex + 1) * batchSize,
+              },
             });
 
             batchesProcessedRef.current += 1;
           }
         } else {
           await props.createNFTFunctions.erc1155.setClaimConditions({
-            nftCollectionInfo: collectionInfo,
-            nftBatch: nfts,
+            values: formValues,
+            batch: {
+              startIndex: 0,
+              endIndex: formValues.nfts.length - 1,
+            },
           });
         }
       }

apps/dashboard/src/components/shared/FileInput.tsx

@@ -26,7 +26,6 @@ interface IFileInputProps {
   children?: React.ReactNode;
   className?: string;
   disableHelperText?: boolean;
-  // fileUrl?: string;
   client: ThirdwebClient;
 }