Skip to content

Commit 9b17b81

Browse files
committed
[FIX] charts: calendar charts cannot be aggregated
Since the big chart datasource refactor, the `aggregate` checkbox appeared in the calendar chart side panels, even though it's not an option for calendar charts (they are always aggregated). Task: 6501177
1 parent 3f6b03d commit 9b17b81

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

src/components/side_panel/chart/calendar_chart/calendar_chart_config_panel.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
updateChart="props.updateChart"
88
canUpdateChart="props.canUpdateChart"
99
onErrorMessagesChanged.bind="onErrorMessagesChanged"
10-
getLabelRangeOptions.bind="getLabelRangeOptions"
10+
getLabelRangeOptions="() => []"
1111
labelRangeTitle.translate="Date range"
1212
/>
1313

tests/figures/chart/calendar/calendar_chart_panel_component.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ describe("Calendar chart side panel", () => {
6161
expect(".o-data-labels input").toHaveValue("A1:A3");
6262
expect(".o-horizontal-group-by").toHaveText("Day of Week");
6363
expect(".o-vertical-group-by").toHaveText("Month");
64+
expect('input[name="aggregated"]').toHaveCount(0);
6465
});
6566

6667
test("Only one data range is enabled", async () => {

0 commit comments

Comments
 (0)