From 930bd99dacb7d55ec9dec9ec382ca34beb74e874 Mon Sep 17 00:00:00 2001
From: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com>
Date: Wed, 31 Jan 2024 20:21:08 +0530
Subject: [PATCH] fix(ui): schedule option for DataInsightReport Application
 (#14966)

---
 .../pages/AppInstall/AppInstall.component.tsx | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/AppInstall/AppInstall.component.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/AppInstall/AppInstall.component.tsx
index c5657280964..a4306e2fdcb 100644
--- a/openmetadata-ui/src/main/resources/ui/src/pages/AppInstall/AppInstall.component.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/pages/AppInstall/AppInstall.component.tsx
@@ -60,11 +60,6 @@ const AppInstall = () => {
   const [appConfiguration, setAppConfiguration] = useState();
   const [jsonSchema, setJsonSchema] = useState<RJSFSchema>();
 
-  const isExternalApp = useMemo(
-    () => appData?.appType === AppType.External,
-    [appData]
-  );
-
   const stepperList = useMemo(
     () =>
       !appData?.allowConfiguration
@@ -126,6 +121,16 @@ const AppInstall = () => {
     setActiveServiceStep(3);
   };
 
+  const initialOptions = useMemo(() => {
+    if (appData?.name === 'DataInsightsReportApplication') {
+      return ['Week'];
+    } else if (appData?.appType === AppType.External) {
+      return ['Day'];
+    }
+
+    return undefined;
+  }, [appData?.name, appData?.appType]);
+
   const RenderSelectedTab = useCallback(() => {
     if (!appData || !jsonSchema) {
       return <></>;
@@ -170,7 +175,7 @@ const AppInstall = () => {
             <Typography.Title level={5}>{t('label.schedule')}</Typography.Title>
             <TestSuiteScheduler
               isQuartzCron
-              includePeriodOptions={isExternalApp ? ['Day'] : undefined}
+              includePeriodOptions={initialOptions}
               initialData={getIngestionFrequency(PipelineType.Application)}
               onCancel={() =>
                 setActiveServiceStep(appData.allowConfiguration ? 2 : 1)
@@ -182,7 +187,7 @@ const AppInstall = () => {
       default:
         return <></>;
     }
-  }, [activeServiceStep, appData, jsonSchema, isExternalApp]);
+  }, [activeServiceStep, appData, jsonSchema, initialOptions]);
 
   useEffect(() => {
     fetchAppDetails();