[ENHANCEMENT] TimeSeriesChart: support migration of unit overrides (#616)

AntoineThebaud · web-flow · commit 80d7feb36a80 · 2026-03-27T15:10:15.000Z
* [ENHANCEMENT] TimeSeriesChart: support migration of unit overrides

Signed-off-by: AntoineThebaud &lt;antoine.thebaud@yahoo.fr&gt;

* cover fallback behavior (no format appended)

Signed-off-by: AntoineThebaud &lt;antoine.thebaud@yahoo.fr&gt;

---------

Signed-off-by: AntoineThebaud &lt;antoine.thebaud@yahoo.fr&gt;
diff --git a/timeserieschart/schemas/migrate/migrate.cue b/timeserieschart/schemas/migrate/migrate.cue
@@ -212,6 +212,12 @@ spec: {
 					][0]
 					areaOpacity: #queryFillOpacity / 100
 				}
+				if property.id == "unit" {
+					#queryUnit: *commonMigrate.#mapping.unit[property.value] | null
+					if #queryUnit != null {
+						format: unit: #queryUnit
+					}
+				}
 			}
 		},
 	]
diff --git a/timeserieschart/schemas/migrate/tests/multiple-overrides/expected.json b/timeserieschart/schemas/migrate/tests/multiple-overrides/expected.json
@@ -46,7 +46,10 @@
         "queryIndex": 4,
         "colorMode": "fixed",
         "colorValue": "#fade2a",
-        "lineStyle": "dashed"
+        "lineStyle": "dashed",
+        "format": {
+          "unit": "requests/sec"
+        }
       }
     ]
   }
diff --git a/timeserieschart/schemas/migrate/tests/multiple-overrides/input.json b/timeserieschart/schemas/migrate/tests/multiple-overrides/input.json
@@ -128,7 +128,7 @@
           },
           {
             "id": "unit",
-            "value": "tps"
+            "value": "reqps"
           },
           {
             "id": "custom.lineStyle",

Original file line number	Diff line number	Diff line change
`@@ -212,6 +212,12 @@ spec: {`
`212`	`212`	`][0]`
`213`	`213`	`areaOpacity: #queryFillOpacity / 100`
`214`	`214`	`}`
	`215`	`+ if property.id == "unit" {`
	`216`	`+ #queryUnit: *commonMigrate.#mapping.unit[property.value] \| null`
	`217`	`+ if #queryUnit != null {`
	`218`	`+ format: unit: #queryUnit`
	`219`	`+ }`
	`220`	`+ }`
`215`	`221`	`}`
`216`	`222`	`},`
`217`	`223`	`]`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,10 @@`
`46`	`46`	`"queryIndex": 4,`
`47`	`47`	`"colorMode": "fixed",`
`48`	`48`	`"colorValue": "#fade2a",`
`49`		`- "lineStyle": "dashed"`
	`49`	`+ "lineStyle": "dashed",`
	`50`	`+ "format": {`
	`51`	`+ "unit": "requests/sec"`
	`52`	`+ }`
`50`	`53`	`}`
`51`	`54`	`]`
`52`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@`
`128`	`128`	`},`
`129`	`129`	`{`
`130`	`130`	`"id": "unit",`
`131`		`- "value": "tps"`
	`131`	`+ "value": "reqps"`
`132`	`132`	`},`
`133`	`133`	`{`
`134`	`134`	`"id": "custom.lineStyle",`