[Feature]: Heatmap 2way sync (#1031)

* Heatmap doesnt render if more than 1 metric or 2 dimensions are added Signed-off-by: Dipra Aich <[email protected]> * Test cases resolution Signed-off-by: Dipra Aich <[email protected]> * Eliminated unnecessary commented code Signed-off-by: Dipra Aich <[email protected]> Signed-off-by: Dipra Aich <[email protected]>
opensearch-project · Sep 19, 2022 · 1c2b396 · 1c2b396
1 parent 8abc193
commit 1c2b396
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/public/components/visualizations/charts/maps/heatmap.tsx b/public/components/visualizations/charts/maps/heatmap.tsx
@@ -37,8 +37,9 @@ export const HeatMap = ({ visualizations, layout, config }: any) => {
     isEmpty(zMetrics) ||
     isEmpty(data[xaxisField.label]) ||
     isEmpty(data[yaxisField.label]) ||
-    isEmpty(data[`${zMetrics.aggregation}(${zMetrics.name})`])
-    // indexOf(NUMERICAL_FIELDS, zMetrics.type) < 0
+    isEmpty(data[`${zMetrics.aggregation}(${zMetrics.name})`]) ||
+    dataConfig?.dimensions.length > 2 ||
+    dataConfig?.metrics.length > 1
   )
     return <EmptyPlaceholder icon={visualizations?.vis?.icontype} />;