You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ru...@apache.org on 2022/08/09 16:57:16 UTC

[superset] 01/01: fix: making crossFilter.chartsInScope optional

This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch fixing-chart-crossfilter-indicator
in repository https://gitbox.apache.org/repos/asf/superset.git

commit f464a0ed71be68a41646dde5344837e6bb0459c2
Author: Evan Rusackas <ev...@preset.io>
AuthorDate: Tue Aug 9 10:57:01 2022 -0600

    fix: making crossFilter.chartsInScope optional
---
 superset-frontend/src/dashboard/components/FiltersBadge/selectors.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset-frontend/src/dashboard/components/FiltersBadge/selectors.ts b/superset-frontend/src/dashboard/components/FiltersBadge/selectors.ts
index 880a6d33e7..855ca303db 100644
--- a/superset-frontend/src/dashboard/components/FiltersBadge/selectors.ts
+++ b/superset-frontend/src/dashboard/components/FiltersBadge/selectors.ts
@@ -294,7 +294,7 @@ export const selectNativeIndicatorsForChart = (
     const dashboardLayoutValues = Object.values(dashboardLayout);
     crossFilterIndicators = Object.values(chartConfiguration)
       .filter(chartConfig =>
-        chartConfig.crossFilters.chartsInScope.includes(chartId),
+        chartConfig.crossFilters?.chartsInScope?.includes(chartId),
       )
       .map(chartConfig => {
         const filterState = dataMask[chartConfig.id]?.filterState;