fix: only set stripe app data if it requires payment (calcom#11623)

Co-authored-by: alannnc <[email protected]>
hm-edu · Sep 29, 2023 · e568d70 · e568d70
1 parent 41b6377
commit e568d70
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/packages/app-store/stripepayment/components/EventTypeAppCardInterface.tsx b/packages/app-store/stripepayment/components/EventTypeAppCardInterface.tsx
@@ -48,13 +48,15 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
       .trim();
 
   useEffect(() => {
-    if (!getAppData("currency")) {
-      setAppData("currency", currencyOptions[0].value);
+    if (requirePayment) {
+      if (!getAppData("currency")) {
+        setAppData("currency", currencyOptions[0].value);
+      }
+      if (!getAppData("paymentOption")) {
+        setAppData("paymentOption", paymentOptions[0].value);
+      }
     }
-    if (!getAppData("paymentOption")) {
-      setAppData("paymentOption", paymentOptions[0].value);
-    }
-  }, []);
+  }, [requirePayment, getAppData, setAppData]);
 
   return (
     <AppCard