[SDK/Dashboard] fix: ERC1155 single phase claim conditions (#6655)

kumaryash90 · kumaryash90 · commit 59be4ea2fbf2 · 2025-04-05T02:26:03.000Z
&lt;!--

## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes"

If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000):

## Notes for the reviewer

Anything important to call out? Be sure to also clarify these in your comments.

## How to test

Unit tests, playground, etc.

--&gt;

&lt;!-- start pr-codex --&gt;

---

## PR-Codex overview
This PR focuses on adding single phase functionality for `erc1155` claim conditions in the `thirdweb` dashboard and related utilities, enhancing the ability to manage claim eligibility and conditions for tokens.

### Detailed summary
- Introduced `isMultiphase` and `isSinglePhase` properties in various components.
- Updated `setClaimPhasesTx` to handle single phase options.
- Modified `getClaimConditions` to support both multi-phase and single-phase claims.
- Enhanced `resetClaimEligibility` to work with single phase drops.
- Updated tests to validate new single phase functionality.

&gt; ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

&lt;!-- end pr-codex --&gt;
diff --git a/.changeset/blue-geese-refuse.md b/.changeset/blue-geese-refuse.md
@@ -0,0 +1,5 @@
+---
+"thirdweb": patch
+---
+
+Single phase functionality for erc1155
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions-form/hooks.ts b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions-form/hooks.ts
@@ -126,7 +126,7 @@ export async function getClaimPhasesInLegacyFormat(
 type PhaseInput = z.input<typeof LegacyClaimConditionInputSchema>;
 
 export function setClaimPhasesTx(
-  baseOptions: BaseTransactionOptions<Options>,
+  baseOptions: BaseTransactionOptions<Options> & { isSinglePhase?: boolean },
   rawPhases: PhaseInput[],
 ) {
   const phases = rawPhases.map((phase) => {
@@ -172,6 +172,7 @@ export function setClaimPhasesTx(
         contract: baseOptions.contract,
         phases,
         tokenId: baseOptions.tokenId,
+        singlePhaseDrop: baseOptions.isSinglePhase,
       });
   }
 }
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions-form/index.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions-form/index.tsx
@@ -226,7 +226,10 @@ export const ClaimConditionsForm: React.FC<ClaimConditionsFormProps> = ({
       ? { type: "erc20", decimals: tokenDecimals.data }
       : isErc721
         ? { type: "erc721" }
-        : { type: "erc1155", tokenId: BigInt(tokenId || 0) }),
+        : {
+            type: "erc1155",
+            tokenId: BigInt(tokenId || 0),
+          }),
   });
 
   const transformedQueryData = useMemo(() => {
@@ -370,6 +373,7 @@ export const ClaimConditionsForm: React.FC<ClaimConditionsFormProps> = ({
       const tx = setClaimPhasesTx(
         {
           contract,
+          isSinglePhase: !isMultiPhase,
           ...(isErc20
             ? { type: "erc20", decimals: tokenDecimals.data }
             : isErc721
@@ -635,6 +639,7 @@ export const ClaimConditionsForm: React.FC<ClaimConditionsFormProps> = ({
                   isErc20={isErc20}
                   contract={contract}
                   tokenId={tokenId}
+                  isMultiphase={isMultiPhase}
                 />
               )}
             </div>
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/claim-conditions.tsx
@@ -10,13 +10,15 @@ interface ClaimConditionsProps {
   isColumn?: true;
   isERC20: boolean;
   twAccount: Account | undefined;
+  isMultiphase: boolean;
 }
 export const ClaimConditions: React.FC<ClaimConditionsProps> = ({
   contract,
   tokenId,
   isColumn,
   isERC20,
   twAccount,
+  isMultiphase,
 }) => {
   return (
     <div className="flex w-full flex-col gap-6">
@@ -37,8 +39,7 @@ export const ClaimConditions: React.FC<ClaimConditionsProps> = ({
         contract={contract}
         tokenId={tokenId}
         isColumn={isColumn}
-        // always multi phase!
-        isMultiPhase={true}
+        isMultiPhase={isMultiphase}
       />
     </div>
   );
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/reset-claim-eligibility.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/_components/claim-conditions/reset-claim-eligibility.tsx
@@ -18,13 +18,15 @@ interface ResetClaimEligibilityProps {
   contract: ThirdwebContract;
   tokenId?: string;
   twAccount: Account | undefined;
+  isMultiphase: boolean;
 }
 
 export const ResetClaimEligibility: React.FC<ResetClaimEligibilityProps> = ({
   contract,
   tokenId,
   isErc20,
   twAccount,
+  isMultiphase,
 }) => {
   const trackEvent = useTrack();
 
@@ -55,6 +57,7 @@ export const ResetClaimEligibility: React.FC<ResetClaimEligibilityProps> = ({
           return ERC1155Ext.resetClaimEligibility({
             contract,
             tokenId: BigInt(tokenId),
+            singlePhaseDrop: !isMultiphase,
           });
         }
         // assume erc 721
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/claim-conditions/ClaimConditions.client.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/claim-conditions/ClaimConditions.client.tsx
@@ -36,6 +36,7 @@ export function ClaimConditionsClient(props: {
       contract={props.contract}
       isERC20={supportedERCs.isERC20}
       twAccount={props.twAccount}
+      isMultiphase={true}
     />
   );
 }
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/claim-conditions/page.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/claim-conditions/page.tsx
@@ -41,6 +41,7 @@ export default async function Page(props: {
       contract={contract}
       isERC20={supportedERCs.isERC20}
       twAccount={account}
+      isMultiphase={true}
     />
   );
 }
diff --git a/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/[tokenId]/useNftDrawerTabs.tsx b/apps/dashboard/src/app/(dashboard)/(chain)/[chain_id]/[contractAddress]/nfts/[tokenId]/useNftDrawerTabs.tsx
@@ -68,7 +68,6 @@ export function useNFTDrawerTabs({
     const hasERC1155ClaimConditions = (() => {
       return [
         // reads
-        ERC1155Ext.isGetClaimConditionByIdSupported(functionSelectors),
         ERC1155Ext.isGetClaimConditionsSupported(functionSelectors),
         ERC1155Ext.isGetActiveClaimConditionSupported(functionSelectors),
         // writes
@@ -77,6 +76,10 @@ export function useNFTDrawerTabs({
       ].every(Boolean);
     })();
 
+    const isERC1155Multiphase = (() => {
+      return ERC1155Ext.isGetClaimConditionByIdSupported(functionSelectors);
+    })();
+
     const isBurnable = (() => {
       if (isERC721) {
         return ERC721Ext.isBurnSupported(functionSelectors);
@@ -130,6 +133,7 @@ export function useNFTDrawerTabs({
               tokenId={tokenId}
               isColumn
               twAccount={twAccount}
+              isMultiphase={isERC1155Multiphase}
             />
           ),
         },
diff --git a/packages/thirdweb/scripts/generate/abis/erc1155/IDropSinglePhase1155.json b/packages/thirdweb/scripts/generate/abis/erc1155/IDropSinglePhase1155.json
@@ -1,7 +1,7 @@
 [
   "function claim(address receiver, uint256 tokenId, uint256 quantity, address currency, uint256 pricePerToken, (bytes32[] proof, uint256 quantityLimitPerWallet, uint256 pricePerToken, address currency) allowlistProof, bytes data) payable",
   "function setClaimConditions(uint256 tokenId, (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) phase, bool resetClaimEligibility)",
-  "function claimCondition(uint256 tokenId) view returns ((uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) condition)",
+  "function claimCondition(uint256 tokenId) view returns (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata)",
   "event ClaimConditionUpdated(uint256 indexed tokenId, (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) condition, bool resetEligibility)",
   "event TokensClaimed(address indexed claimer, address indexed receiver, uint256 indexed tokenId, uint256 quantityClaimed)"
 ]
diff --git a/packages/thirdweb/src/exports/extensions/erc1155.ts b/packages/thirdweb/src/exports/extensions/erc1155.ts
@@ -112,6 +112,7 @@ export {
   type CanClaimParams,
   type CanClaimResult,
 } from "../../extensions/erc1155/drops/read/canClaim.js";
+export { claimCondition } from "../../extensions/erc1155/__generated__/IDropSinglePhase1155/read/claimCondition.js";
 
 // WRITE
 export {
diff --git a/packages/thirdweb/src/extensions/erc1155/__generated__/IDropSinglePhase1155/read/claimCondition.ts b/packages/thirdweb/src/extensions/erc1155/__generated__/IDropSinglePhase1155/read/claimCondition.ts
diff --git a/packages/thirdweb/src/extensions/erc1155/customDrop1155.test.ts b/packages/thirdweb/src/extensions/erc1155/customDrop1155.test.ts
@@ -7,13 +7,16 @@ import { TEST_CLIENT } from "../../../test/src/test-clients.js";
 import {
   TEST_ACCOUNT_B,
   TEST_ACCOUNT_C,
+  TEST_ACCOUNT_D,
 } from "../../../test/src/test-wallets.js";
 import { type ThirdwebContract, getContract } from "../../contract/contract.js";
 import { sendAndConfirmTransaction } from "../../transaction/actions/send-and-confirm-transaction.js";
 import { deployContractfromDeployMetadata } from "../prebuilts/deploy-published.js";
 import { balanceOf } from "./__generated__/IERC1155/read/balanceOf.js";
 import { nextTokenIdToMint } from "./__generated__/IERC1155Enumerable/read/nextTokenIdToMint.js";
+import { getClaimConditions } from "./drops/read/getClaimConditions.js";
 import { claimTo } from "./drops/write/claimTo.js";
+import { resetClaimEligibility } from "./drops/write/resetClaimEligibility.js";
 import { setClaimConditions } from "./drops/write/setClaimConditions.js";
 import { getNFT } from "./read/getNFT.js";
 import { lazyMint } from "./write/lazyMint.js";
@@ -92,6 +95,14 @@ describe.runIf(process.env.TW_SECRET_KEY)(
         }),
         account: TEST_ACCOUNT_C,
       });
+
+      const condition = await getClaimConditions({
+        contract,
+        tokenId: 0n,
+      });
+      expect(condition.length).to.eq(1);
+      expect(condition[0]?.maxClaimableSupply).to.eq(10n);
+
       const claimTx = claimTo({
         contract,
         to: TEST_ACCOUNT_C.address,
@@ -166,5 +177,80 @@ describe.runIf(process.env.TW_SECRET_KEY)(
           chainId: ${contract.chain.id}]
         `);
     });
+
+    it("should reset claim eligibility", async () => {
+      await sendAndConfirmTransaction({
+        transaction: setClaimConditions({
+          contract,
+          phases: [
+            {
+              startTime: new Date(0),
+              maxClaimableSupply: 10n,
+              maxClaimablePerWallet: 1n,
+            },
+          ],
+          tokenId: 0n,
+          singlePhaseDrop: true,
+        }),
+        account: TEST_ACCOUNT_C,
+      });
+      // claim one token
+      const claimTx = claimTo({
+        contract,
+        to: TEST_ACCOUNT_D.address,
+        tokenId: 0n,
+        quantity: 1n,
+        singlePhaseDrop: true,
+      });
+      await sendAndConfirmTransaction({
+        transaction: claimTx,
+        account: TEST_ACCOUNT_D,
+      });
+      await expect(
+        balanceOf({ contract, owner: TEST_ACCOUNT_D.address, tokenId: 0n }),
+      ).resolves.toBe(1n);
+
+      // attempt to claim another token (this should fail)
+      await expect(
+        sendAndConfirmTransaction({
+          transaction: claimTo({
+            tokenId: 0n,
+            contract,
+            to: TEST_ACCOUNT_D.address,
+            quantity: 1n,
+          }),
+          account: TEST_ACCOUNT_D,
+        }),
+      ).rejects.toThrowErrorMatchingInlineSnapshot(`
+        [TransactionError: Error - !Qty
+
+        contract: ${contract.address}
+        chainId: ${contract.chain.id}]
+      `);
+
+      // reset claim eligibility
+      await sendAndConfirmTransaction({
+        transaction: resetClaimEligibility({
+          tokenId: 0n,
+          contract,
+          singlePhaseDrop: true,
+        }),
+        account: TEST_ACCOUNT_C,
+      });
+      // attempt to claim another token (this should succeed)
+      await sendAndConfirmTransaction({
+        transaction: claimTo({
+          tokenId: 0n,
+          contract,
+          to: TEST_ACCOUNT_D.address,
+          quantity: 1n,
+        }),
+        account: TEST_ACCOUNT_D,
+      });
+      // check that the account has claimed two tokens
+      await expect(
+        balanceOf({ tokenId: 0n, contract, owner: TEST_ACCOUNT_D.address }),
+      ).resolves.toBe(2n);
+    });
   },
 );
diff --git a/packages/thirdweb/src/extensions/erc1155/drops/read/getActiveClaimCondition.ts b/packages/thirdweb/src/extensions/erc1155/drops/read/getActiveClaimCondition.ts
diff --git a/packages/thirdweb/src/extensions/erc1155/drops/read/getClaimConditions.ts b/packages/thirdweb/src/extensions/erc1155/drops/read/getClaimConditions.ts
diff --git a/packages/thirdweb/src/extensions/erc1155/drops/write/resetClaimEligibility.ts b/packages/thirdweb/src/extensions/erc1155/drops/write/resetClaimEligibility.ts
diff --git a/packages/thirdweb/src/utils/extensions/drops/get-claim-params.ts b/packages/thirdweb/src/utils/extensions/drops/get-claim-params.ts

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"thirdweb": patch
 +---
++
 +Single phase functionality for erc1155
Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ export function ClaimConditionsClient(props: {`
`36`	`36`	`contract={props.contract}`
`37`	`37`	`isERC20={supportedERCs.isERC20}`
`38`	`38`	`twAccount={props.twAccount}`
	`39`	`+ isMultiphase={true}`
`39`	`40`	`/>`
`40`	`41`	`);`
`41`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,7 @@ export default async function Page(props: {`
`41`	`41`	`contract={contract}`
`42`	`42`	`isERC20={supportedERCs.isERC20}`
`43`	`43`	`twAccount={account}`
	`44`	`+ isMultiphase={true}`
`44`	`45`	`/>`
`45`	`46`	`);`
`46`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`[`
`2`	`2`	`"function claim(address receiver, uint256 tokenId, uint256 quantity, address currency, uint256 pricePerToken, (bytes32[] proof, uint256 quantityLimitPerWallet, uint256 pricePerToken, address currency) allowlistProof, bytes data) payable",`
`3`	`3`	`"function setClaimConditions(uint256 tokenId, (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) phase, bool resetClaimEligibility)",`
`4`		`- "function claimCondition(uint256 tokenId) view returns ((uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) condition)",`
	`4`	`+ "function claimCondition(uint256 tokenId) view returns (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata)",`
`5`	`5`	`"event ClaimConditionUpdated(uint256 indexed tokenId, (uint256 startTimestamp, uint256 maxClaimableSupply, uint256 supplyClaimed, uint256 quantityLimitPerWallet, bytes32 merkleRoot, uint256 pricePerToken, address currency, string metadata) condition, bool resetEligibility)",`
`6`	`6`	`"event TokensClaimed(address indexed claimer, address indexed receiver, uint256 indexed tokenId, uint256 quantityClaimed)"`
`7`	`7`	`]`