From 53cdbd8a2df451dfc440fffaea095a14f8414866 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torkel=20=C3=96degaard?= Date: Thu, 5 Dec 2024 12:50:21 +0100 Subject: [PATCH] Dashboard: Fixes issue with compatability of old DashboardModel.annotations (#97328) * Dashboard: Fixes issue with compatability of old DashboardModel.annotations * let's lint this thing --------- Co-authored-by: Dominik Prokop --- .../DashboardModelCompatibilityWrapper.test.ts | 14 ++++---------- .../utils/DashboardModelCompatibilityWrapper.ts | 7 ++++--- 2 files changed, 8 insertions(+), 13 deletions(-) diff --git a/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.test.ts b/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.test.ts index d8a9b152b03..cd43c5f6975 100644 --- a/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.test.ts +++ b/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.test.ts @@ -1,18 +1,12 @@ import { TimeRangeUpdatedEvent } from '@grafana/runtime'; -import { - behaviors, - SceneQueryRunner, - SceneTimeRange, - VizPanel, - SceneDataTransformer, - SceneDataLayerSet, -} from '@grafana/scenes'; +import { behaviors, SceneQueryRunner, SceneTimeRange, VizPanel, SceneDataTransformer } from '@grafana/scenes'; import { DashboardCursorSync } from '@grafana/schema'; import { SHARED_DASHBOARD_QUERY } from 'app/plugins/datasource/dashboard'; import { AlertStatesDataLayer } from '../scene/AlertStatesDataLayer'; import { DashboardAnnotationsDataLayer } from '../scene/DashboardAnnotationsDataLayer'; import { DashboardControls } from '../scene/DashboardControls'; +import { DashboardDataLayerSet } from '../scene/DashboardDataLayerSet'; import { DashboardScene } from '../scene/DashboardScene'; import { DefaultGridLayoutManager } from '../scene/layout-default/DefaultGridLayoutManager'; import { NEW_LINK } from '../settings/links/utils'; @@ -150,8 +144,8 @@ function setup() { weekStart: 'friday', timeZone: 'America/New_York', }), - $data: new SceneDataLayerSet({ - layers: [ + $data: new DashboardDataLayerSet({ + annotationLayers: [ new DashboardAnnotationsDataLayer({ key: `annotations-test`, query: { diff --git a/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.ts b/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.ts index 9140b9ce6ae..e957ff1760b 100644 --- a/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.ts +++ b/public/app/features/dashboard-scene/utils/DashboardModelCompatibilityWrapper.ts @@ -2,8 +2,9 @@ import { Subscription } from 'rxjs'; import { AnnotationQuery, DashboardCursorSync, dateTimeFormat, DateTimeInput, EventBusSrv } from '@grafana/data'; import { TimeRangeUpdatedEvent } from '@grafana/runtime'; -import { behaviors, SceneDataLayerSet, sceneGraph, SceneObject, VizPanel } from '@grafana/scenes'; +import { behaviors, sceneGraph, SceneObject, VizPanel } from '@grafana/scenes'; +import { DashboardDataLayerSet } from '../scene/DashboardDataLayerSet'; import { DashboardScene } from '../scene/DashboardScene'; import { dataLayersToAnnotations } from '../serialization/dataLayersToAnnotations'; @@ -104,8 +105,8 @@ export class DashboardModelCompatibilityWrapper { public get annotations(): { list: AnnotationQuery[] } { const annotations: { list: AnnotationQuery[] } = { list: [] }; - if (this._scene.state.$data instanceof SceneDataLayerSet) { - annotations.list = dataLayersToAnnotations(this._scene.state.$data.state.layers); + if (this._scene.state.$data instanceof DashboardDataLayerSet) { + annotations.list = dataLayersToAnnotations(this._scene.state.$data.state.annotationLayers); } return annotations;