From e568d70e651ad87ae20d1689127534a1c6de7f00 Mon Sep 17 00:00:00 2001
From: Roland <33993199+rolznz@users.noreply.github.com>
Date: Sat, 30 Sep 2023 01:49:52 +0700
Subject: [PATCH] fix: only set stripe app data if it requires payment (#11623)

Co-authored-by: alannnc <alannnc@gmail.com>
---
 .../components/EventTypeAppCardInterface.tsx       | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/packages/app-store/stripepayment/components/EventTypeAppCardInterface.tsx b/packages/app-store/stripepayment/components/EventTypeAppCardInterface.tsx
index 4da5ab4d58..ba0dc68087 100644
--- 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