allow using approveHandler even if 2612 supported

0xPolygonID · Oct 30, 2024 · a7914aa · a7914aa
1 parent 830f705
commit a7914aa
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/iden3comm/handlers/payment.ts b/src/iden3comm/handlers/payment.ts
@@ -658,13 +658,11 @@ export class PaymentHandler
     paymentHandler: (data: Iden3PaymentRailsERC20RequestV1) => Promise<string>,
     approveHandler?: (data: Iden3PaymentRailsERC20RequestV1) => Promise<string>
   ): Promise<Iden3PaymentRailsERC20V1> {
-    if (!data.features?.includes(PaymentFeatures.EIP_2612)) {
-      if (!approveHandler) {
-        throw new Error(
-          `please provide erc20TokenApproveHandler in context for ERC-20 payment type`
-        );
-      }
+    if (!data.features?.includes(PaymentFeatures.EIP_2612) && !approveHandler) {
+      throw new Error(`please provide erc20TokenApproveHandler in context for ERC-20 payment type`);
+    }
 
+    if (approveHandler) {
       await approveHandler(data);
     }