[TOOL-2924] Catch error on fail-to-load metadata (#5892)

TOOL-2924
thirdweb-dev · Jan 6, 2025 · 9bc7884 · 9bc7884
1 parent 6bbfc4d
commit 9bc7884
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...rc/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/components/reveal-button.tsx b/...rc/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/components/reveal-button.tsx
@@ -70,7 +70,7 @@ export const NFTRevealButton: React.FC<NFTRevealButtonProps> = ({
     );
   }
 
-  return batchesQuery.data?.length ? (
+  return (
     <MinterOnly contract={contract}>
       <Sheet open={open} onOpenChange={setOpen}>
         <SheetTrigger asChild>
@@ -168,5 +168,5 @@ export const NFTRevealButton: React.FC<NFTRevealButtonProps> = ({
         </SheetContent>
       </Sheet>
     </MinterOnly>
-  ) : null;
+  );
 };
diff --git a/packages/thirdweb/src/extensions/erc721/lazyMinting/read/getBatchesToReveal.ts b/packages/thirdweb/src/extensions/erc721/lazyMinting/read/getBatchesToReveal.ts
@@ -15,7 +15,7 @@ import * as TokenURI from "../../__generated__/IERC721A/read/tokenURI.js";
 export interface BatchToReveal {
   batchId: bigint;
   batchUri: string;
-  placeholderMetadata: NFTMetadata;
+  placeholderMetadata: NFTMetadata | undefined;
 }
 
 /**
@@ -82,7 +82,7 @@ export async function getBatchesToReveal(
         tokenId: BigInt(i),
         tokenUri: uri,
         client: options.contract.client,
-      });
+      }).catch(() => undefined);
     }),
   );