Add descriptor metadata

- Fixes some issues on Enterprise where query or session information was dropped
deephaven · Jul 17, 2024 · bd18082 · bd18082
1 parent ab36f2a
commit bd18082
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/dashboard-core-plugins/src/ChartPanelPlugin.tsx b/packages/dashboard-core-plugins/src/ChartPanelPlugin.tsx
@@ -75,6 +75,7 @@ async function createChartModel(
 
     if (metadata.type === dh.VariableType.FIGURE) {
       const descriptor = {
+        ...metadata,
         name: figureName,
         type: dh.VariableType.FIGURE,
       };
@@ -86,8 +87,8 @@ async function createChartModel(
     return ChartModelFactory.makeModel(dh, settings, figure);
   }
 
-  // TODO #1940: This won't work on Enterprise. We will need to refactor how ChartBuilder works.
   const descriptor = {
+    ...metadata,
     name: tableName,
     type: dh.VariableType.TABLE,
   };

diff --git a/packages/dashboard-core-plugins/src/panels/IrisGridPanel.tsx b/packages/dashboard-core-plugins/src/panels/IrisGridPanel.tsx
@@ -729,7 +729,7 @@ export class IrisGridPanel extends PureComponent<
     this.setState(
       () => null,
       () => {
-        const { glEventHub, inputFilters } = this.props;
+        const { glEventHub, inputFilters, metadata } = this.props;
         const table = this.getTableName();
         const { panelState } = this.state;
         const sourcePanelId = LayoutUtils.getIdFromPanel(this);
@@ -743,6 +743,7 @@ export class IrisGridPanel extends PureComponent<
         }
         glEventHub.emit(IrisGridEvent.CREATE_CHART, {
           metadata: {
+            ...metadata,
             settings,
             sourcePanelId,
             table,