Dashboard: Default weekStart to an empty string (#114932)

* Default clien scene-based logic to empty string to match backend + non-scene logic

* re-gen snapshots
This commit is contained in:
Kristina Demeshchik
2025-12-08 10:49:36 -05:00
committed by GitHub
parent b50cf6e067
commit fef6196195
2 changed files with 4 additions and 5 deletions
@@ -709,6 +709,7 @@ exports[`transformSceneToSaveModel Given a scene with rows Should transform back
"title": "Repeating rows",
"uid": "Repeating-rows-uid",
"version": 1,
"weekStart": "",
}
`;
@@ -143,6 +143,7 @@ export function transformSceneToSaveModel(scene: DashboardScene, isSnapshot = fa
description: state.description || undefined,
uid: state.uid,
id: state.id,
editable: state.editable,
preload: state.preload,
time: {
from: timeRange.from,
@@ -158,7 +159,7 @@ export function transformSceneToSaveModel(scene: DashboardScene, isSnapshot = fa
},
version: state.version,
fiscalYearStartMonth: timeRange.fiscalYearStartMonth,
weekStart: timeRange.weekStart,
weekStart: timeRange.weekStart ?? '',
tags: state.tags,
links: state.links,
graphTooltip,
@@ -170,10 +171,7 @@ export function transformSceneToSaveModel(scene: DashboardScene, isSnapshot = fa
};
// Only add optional fields if they are explicitly set (not default values)
if (state.editable !== undefined) {
dashboard.editable = state.editable;
}
if (timeRange.timeZone !== undefined && timeRange.timeZone !== '') {
if (timeRange.timeZone !== '') {
dashboard.timezone = timeRange.timeZone;
}