Merge pull request #757 from visualize-admin/fix/embed-config-migrations

fix: Migrate chart config in embed mode

CHANGELOG.md

@@ -10,6 +10,7 @@ You can also check the [release page](https://github.com/visualize-admin/visuali
 ## Unreleased
 
 - Charts: fix bugs that caused scatterplot and pie charts to crash in case no categorical dimensions were present in a dataset
+- Embed: introduce chart config migrations to embed mode
 
 ## [3.9.4] - 2022-09-29
 

app/db/config.ts

@@ -1,3 +1,5 @@
+import { migrateChartConfig } from "@/utils/chart-config/versioning";
+
 import { createChartId } from "../utils/create-chart-id";
 
 import { pool } from "./pg-pool";
@@ -37,7 +39,19 @@ export const getConfig = async (
     [key]
   );
 
-  return result.rows[0];
+  const config = result.rows[0];
+
+  if (config && config.data) {
+    return {
+      ...config,
+      data: {
+        ...config.data,
+        chartConfig: migrateChartConfig(config.data.chartConfig),
+      },
+    };
+  }
+
+  return config;
 };
 
 /**

app/pages/v/[chartId].tsx

@@ -15,7 +15,6 @@ import { PublishActions } from "@/components/publish-actions";
 import { Config } from "@/configurator";
 import { getConfig } from "@/db/config";
 import { useLocale } from "@/locales/use-locale";
-import { migrateChartConfig } from "@/utils/chart-config/versioning";
 
 type PageProps =
   | {
@@ -37,18 +36,7 @@ export const getServerSideProps: GetServerSideProps<PageProps> = async ({
 
   if (config && config.data) {
     // TODO validate configuration
-    return {
-      props: {
-        status: "found",
-        config: {
-          ...config,
-          data: {
-            ...config.data,
-            chartConfig: migrateChartConfig(config.data.chartConfig),
-          },
-        },
-      },
-    };
+    return { props: { status: "found", config } };
   }
 
   res.statusCode = 404;