fix: remove annotation layer from query package

packages/superset-ui-chart/src/clients/ChartClient.ts

@@ -6,10 +6,11 @@ import {
   Json,
   SupersetClientClass,
 } from '@superset-ui/connection';
-import { AnnotationLayerMetadata, QueryFormData, Datasource } from '@superset-ui/query';
+import { QueryFormData, Datasource } from '@superset-ui/query';
 import getChartBuildQueryRegistry from '../registries/ChartBuildQueryRegistrySingleton';
 import getChartMetadataRegistry from '../registries/ChartMetadataRegistrySingleton';
 import { QueryData } from '../models/ChartProps';
+import { AnnotationLayerMetadata } from '../types/Annotation';
 
 // This expands to Partial<All> & (union of all possible single-property types)
 type AtLeastOne<All, Each = { [K in keyof All]: Pick<All, K> }> = Partial<All> & Each[keyof Each];
@@ -130,17 +131,23 @@ export default class ChartClient {
   }
 
   loadChartData(input: SliceIdAndOrFormData): Promise<ChartData> {
-    return this.loadFormData(input).then(formData =>
-      Promise.all([
-        this.loadAnnotations(formData.annotation_layers),
-        this.loadDatasource(formData.datasource),
-        this.loadQueryData(formData),
-      ]).then(([annotationData, datasource, queryData]) => ({
-        annotationData,
-        datasource,
-        formData,
-        queryData,
-      })),
+    return this.loadFormData(input).then(
+      (
+        formData: QueryFormData & {
+          // eslint-disable-next-line camelcase
+          annotation_layers?: AnnotationLayerMetadata[];
+        },
+      ) =>
+        Promise.all([
+          this.loadAnnotations(formData.annotation_layers),
+          this.loadDatasource(formData.datasource),
+          this.loadQueryData(formData),
+        ]).then(([annotationData, datasource, queryData]) => ({
+          annotationData,
+          datasource,
+          formData,
+          queryData,
+        })),
     );
   }
 }

packages/superset-ui-query/src/index.ts

@@ -4,7 +4,6 @@ export { default as convertFilter } from './convertFilter';
 export { default as convertMetric } from './convertMetric';
 export { default as DatasourceKey } from './DatasourceKey';
 
-export * from './types/Annotation';
 export * from './types/QueryFormData';
 export * from './types/Column';
 export * from './types/Datasource';

packages/superset-ui-query/src/types/QueryFormData.ts

@@ -2,7 +2,6 @@
 /* eslint-disable import/prefer-default-export */
 // FormData uses snake_cased keys.
 import { MetricKey, AdhocMetric } from './Metric';
-import { AnnotationLayerMetadata } from './Annotation';
 import { TimeRange } from './Time';
 import { AdhocFilter } from './Filter';
 
@@ -47,8 +46,6 @@ export type BaseFormData = {
   row_limit?: number;
   /** The metric used to order timeseries for limiting */
   timeseries_limit_metric?: QueryFormDataMetric;
-
-  annotation_layers?: AnnotationLayerMetadata[];
 } & TimeRange &
   QueryFormDataMetrics;