From b5beb503410fa377f766898a8d39e1b07d26c232 Mon Sep 17 00:00:00 2001
From: Petr Stribny <petr@stribny.name>
Date: Mon, 10 Mar 2025 08:32:32 +0100
Subject: [PATCH] Only list aggregation types that make sense for source table
in chart configuration
---
.../components/data_source/AggregationSeriesForm.vue | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/enterprise/web-frontend/modules/baserow_enterprise/dashboard/components/data_source/AggregationSeriesForm.vue b/enterprise/web-frontend/modules/baserow_enterprise/dashboard/components/data_source/AggregationSeriesForm.vue
index a75936175..d4e9e2c50 100644
--- a/enterprise/web-frontend/modules/baserow_enterprise/dashboard/components/data_source/AggregationSeriesForm.vue
+++ b/enterprise/web-frontend/modules/baserow_enterprise/dashboard/components/data_source/AggregationSeriesForm.vue
@@ -98,7 +98,13 @@ export default {
},
computed: {
groupedAggregationTypes() {
- return this.$registry.getOrderedList('groupedAggregation')
+ const allAggregationTypes =
+ this.$registry.getOrderedList('groupedAggregation')
+ return allAggregationTypes.filter((aggType) => {
+ return this.tableFields.some((tableField) =>
+ aggType.fieldIsCompatible(tableField)
+ )
+ })
},
aggregationTypesAvailableForSelection() {
return this.groupedAggregationTypes.filter(