fix: simple collect module recipient not optional w/ amount

lens-protocol · Jul 4, 2024 · f545645 · f545645
1 parent 1d2122d
commit f545645
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/.changeset/two-avocados-accept.md b/.changeset/two-avocados-accept.md
@@ -0,0 +1,7 @@
+---
+"@lens-protocol/react-native": patch
+"@lens-protocol/react-web": patch
+"@lens-protocol/react": patch
+---
+
+**fix:** Simple Collect Module recipient is not optional when amount is specified.
diff --git a/packages/domain/src/use-cases/publications/OpenActionConfig.ts b/packages/domain/src/use-cases/publications/OpenActionConfig.ts
@@ -42,6 +42,8 @@ export type SimpleCollectActionConfig = {
   referralFee?: number;
   /**
    * The recipient of the collect fee.
+   *
+   * You MUST provide a recipient if you provide an amount.
    */
   recipient?: EvmAddress;
   /**

diff --git a/packages/react/src/transactions/adapters/publications/resolveOpenActionModuleInput.ts b/packages/react/src/transactions/adapters/publications/resolveOpenActionModuleInput.ts
@@ -1,3 +1,4 @@
+import { AddressZero } from '@ethersproject/constants';
 import { OpenActionModuleInput } from '@lens-protocol/api-bindings';
 import { OpenActionConfig, OpenActionType } from '@lens-protocol/domain/use-cases/publications';
 
@@ -15,7 +16,7 @@ export function resolveOpenActionModuleInput(config: OpenActionConfig): OpenActi
 
               referralFee: config.referralFee,
 
-              recipient: config.recipient ?? null,
+              recipient: config.recipient ?? AddressZero,
             }),
 
             collectLimit: config.collectLimit?.toString() ?? null,