feat async download

thoughtspot · Feb 4, 2025 · dc58e9e · dc58e9e
1 parent a00a6ef
commit dc58e9e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/main/custom-chart-context.ts b/src/main/custom-chart-context.ts
@@ -1038,16 +1038,16 @@ export class CustomChartContext {
          */
         this.on(
             TSToChartEvent.DownloadExcelTrigger,
-            (payload: DownloadExcelTriggerPayload) => {
+            async (payload: DownloadExcelTriggerPayload) => {
                 logger.log(
                     'DownloadExcelTrigger event triggered from TS with payload:',
                     payload,
                 );
-                return {
+                return Promise.resolve({
                     fileName: '',
                     error: '',
                     message: 'Download Excel not implemented.',
-                };
+                });
             },
         );
     };

diff --git a/src/react/use-custom-chart-context.tsx b/src/react/use-custom-chart-context.tsx
@@ -112,11 +112,11 @@ export const useChartContext = (
             payload: DownloadExcelTriggerPayload,
         ) => {
             setChartModel(context.getChartModel());
-            return {
+            return Promise.resolve({
                 fileName: '',
                 error: '',
                 message: 'Download Excel not implemented.',
-            };
+            });
         };
         // Register all external event listeners here
         getChartContextValues(context).setOnChartModelUpdate(

diff --git a/src/types/ts-to-chart-event.types.ts b/src/types/ts-to-chart-event.types.ts
@@ -110,7 +110,7 @@ export interface TSToChartEventsPayloadMap {
     ) => void;
     [TSToChartEvent.DownloadExcelTrigger]: (
         payload: DownloadExcelTriggerPayload,
-    ) => DownloadExcelTriggerResponse;
+    ) => Promise<DownloadExcelTriggerResponse>;
 }
 
 /**