diff --git a/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetBarChart/utils/transformGroupByDataToBarChartData.ts b/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetBarChart/utils/transformGroupByDataToBarChartData.ts index 4159416cfd..add9397d65 100644 --- a/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetBarChart/utils/transformGroupByDataToBarChartData.ts +++ b/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetBarChart/utils/transformGroupByDataToBarChartData.ts @@ -95,35 +95,36 @@ export const transformGroupByDataToBarChartData = ({ const queryResultGqlFieldName = getGroupByQueryResultGqlFieldName(objectMetadataItem); const rawResults = groupByData?.[queryResultGqlFieldName]; - const hasNoData = - !isDefined(groupByData) || - !isDefined(rawResults) || - !Array.isArray(rawResults) || - rawResults.length === 0; - - const showXAxis = - hasNoData || - configuration.axisNameDisplay === AxisNameDisplay.X || - configuration.axisNameDisplay === AxisNameDisplay.BOTH; - - const showYAxis = - hasNoData || - configuration.axisNameDisplay === AxisNameDisplay.Y || - configuration.axisNameDisplay === AxisNameDisplay.BOTH; - - const xAxisLabel = - showXAxis && isDefined(groupByFieldX) ? groupByFieldX.label : undefined; - - const yAxisLabel = - showYAxis && isDefined(aggregateField) - ? `${getAggregateOperationLabel(configuration.aggregateOperation)} of ${aggregateField.label}` - : undefined; const layout = configuration.layout === BarChartLayout.HORIZONTAL ? BarChartLayout.HORIZONTAL : BarChartLayout.VERTICAL; + const isHorizontal = layout === BarChartLayout.HORIZONTAL; + + const showCategoryLabel = isHorizontal + ? configuration.axisNameDisplay === AxisNameDisplay.Y || + configuration.axisNameDisplay === AxisNameDisplay.BOTH + : configuration.axisNameDisplay === AxisNameDisplay.X || + configuration.axisNameDisplay === AxisNameDisplay.BOTH; + + const showValueLabel = isHorizontal + ? configuration.axisNameDisplay === AxisNameDisplay.X || + configuration.axisNameDisplay === AxisNameDisplay.BOTH + : configuration.axisNameDisplay === AxisNameDisplay.Y || + configuration.axisNameDisplay === AxisNameDisplay.BOTH; + + const xAxisLabel = + showCategoryLabel && isDefined(groupByFieldX) + ? groupByFieldX.label + : undefined; + + const yAxisLabel = + showValueLabel && isDefined(aggregateField) + ? `${getAggregateOperationLabel(configuration.aggregateOperation)} of ${aggregateField.label}` + : undefined; + if (!isDefined(groupByData)) { return { ...EMPTY_BAR_CHART_RESULT, diff --git a/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetLineChart/utils/transformGroupByDataToLineChartData.ts b/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetLineChart/utils/transformGroupByDataToLineChartData.ts index d16ba62fe8..88d40a5639 100644 --- a/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetLineChart/utils/transformGroupByDataToLineChartData.ts +++ b/packages/twenty-front/src/modules/page-layout/widgets/graph/graphWidgetLineChart/utils/transformGroupByDataToLineChartData.ts @@ -82,19 +82,12 @@ export const transformGroupByDataToLineChartData = ({ const queryResultGqlFieldName = getGroupByQueryResultGqlFieldName(objectMetadataItem); const rawResults = groupByData?.[queryResultGqlFieldName]; - const hasNoData = - !isDefined(groupByData) || - !isDefined(rawResults) || - !Array.isArray(rawResults) || - rawResults.length === 0; const showXAxis = - hasNoData || configuration.axisNameDisplay === AxisNameDisplay.X || configuration.axisNameDisplay === AxisNameDisplay.BOTH; const showYAxis = - hasNoData || configuration.axisNameDisplay === AxisNameDisplay.Y || configuration.axisNameDisplay === AxisNameDisplay.BOTH;