fix: Dashboard shared filter can work on merged cubes

Author: ptbrowne

app/components/dashboard-interactive-filters.tsx

@@ -185,7 +185,7 @@ const DashboardTimeRangeSlider = ({
 
   const mountedForSomeTime = useTimeout(500, mounted);
 
-  if (!filter || !timeRange || filter.type !== "timeRange") {
+  if (!filter || !timeRange || filter.type !== "timeRange" || !filter.active) {
     return null;
   }
 

app/configurator/components/layout-configurator.tsx

@@ -41,6 +41,7 @@ import {
 import {
   Dimension,
   TemporalDimension,
+  isJoinByComponent,
   isTemporalDimension,
 } from "@/domain/data";
 import { useTimeFormatLocale, useTimeFormatUnit } from "@/formatters";
@@ -129,7 +130,16 @@ const LayoutSharedFiltersConfigurator = () => {
   });
 
   const dimensionsByIri = useMemo(() => {
-    return keyBy(data.data?.dataCubesComponents.dimensions, (x) => x.iri);
+    const res: Record<string, Dimension> = {};
+    for (const dim of data.data?.dataCubesComponents.dimensions ?? []) {
+      res[dim.iri] = dim;
+      if (isJoinByComponent(dim)) {
+        for (const o of dim.originalIris) {
+          res[o.dimensionIri] = dim;
+        }
+      }
+    }
+    return res;
   }, [data.data?.dataCubesComponents.dimensions]);
 
   const sharedFiltersByIri = useMemo(() => {

app/stores/interactive-filters.tsx

@@ -12,6 +12,7 @@ import {
   useConfiguratorState,
 } from "@/configurator";
 import { truthy } from "@/domain/types";
+import { getOriginalIris, isJoinById } from "@/graphql/join";
 import {
   createBoundUseStoreWithSelector,
   ExtractState,
@@ -177,7 +178,10 @@ export const getPotentialSharedFilters = (
             : undefined;
         if (field && "componentIri" in field) {
           return {
-            componentIri: field.componentIri as string,
+            /** Unjoined dimension */
+            componentIri: isJoinById(field.componentIri as string)
+              ? getOriginalIris(field.componentIri, config)[0]
+              : field.componentIri,
             chartKey: config.key,
           };
         }