fix(storybook): fix broken Storybook stories during development (apac…

…he#29587) Signed-off-by: hainenber <dotronghai96@gmail.com>
justinpark · Jul 15, 2024 · 462cda4 · 462cda4
1 parent fb15278
commit 462cda4
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 4 deletions.
diff --git a/superset-frontend/packages/superset-ui-demo/.storybook/preview.js b/superset-frontend/packages/superset-ui-demo/.storybook/preview.js
@@ -55,7 +55,6 @@ export const parameters = {
 };
 
 // Superset setup
-
 configure();
 
 // Register color schemes

diff --git a/...rset-ui-demo/storybook/stories/plugins/legacy-preset-chart-nvd3/BoxPlot/stories/basic.tsx b/...rset-ui-demo/storybook/stories/plugins/legacy-preset-chart-nvd3/BoxPlot/stories/basic.tsx
@@ -24,8 +24,8 @@ import data from '../data';
 export const basic = () => (
   <SuperChart
     chartType="box-plot"
-    width={400}
-    height={400}
+    width={800}
+    height={600}
     datasource={dummyDatasource}
     queriesData={[{ data }]}
     formData={{

diff --git a/...hart-echarts/BoxPlot/BoxPolot.stories.tsx → ...chart-echarts/BoxPlot/BoxPlot.stories.tsx b/...hart-echarts/BoxPlot/BoxPolot.stories.tsx → ...chart-echarts/BoxPlot/BoxPlot.stories.tsx
diff --git a/superset-frontend/plugins/legacy-preset-chart-nvd3/src/transformProps.js b/superset-frontend/plugins/legacy-preset-chart-nvd3/src/transformProps.js
@@ -111,7 +111,9 @@ export default function transformProps(chartProps) {
   const data = Array.isArray(rawData)
     ? rawData.map(row => ({
         ...row,
-        values: row.values.map(value => ({ ...value })),
+        values: Array.isArray(row.values)
+          ? row.values.map(value => ({ ...value }))
+          : row.values,
         key: formatLabel(row.key, datasource.verboseMap),
       }))
     : rawData;

diff --git a/superset-frontend/src/components/CertifiedBadge/CertifiedBadge.stories.tsx b/superset-frontend/src/components/CertifiedBadge/CertifiedBadge.stories.tsx
@@ -16,8 +16,11 @@
  * specific language governing permissions and limitations
  * under the License.
  */
+import { configure as configureTranslation } from '@superset-ui/core';
 import CertifiedBadge, { CertifiedBadgeProps } from '.';
 
+configureTranslation();
+
 export default {
   title: 'CertifiedBadgeWithTooltip',
 };

diff --git a/superset-frontend/webpack.config.js b/superset-frontend/webpack.config.js
@@ -75,6 +75,7 @@ if (!isDevMode) {
 const plugins = [
   new webpack.ProvidePlugin({
     process: 'process/browser.js',
+    ...(isDevMode ? { Buffer: ['buffer', 'Buffer'] } : {}), // Fix legacy-plugin-chart-paired-t-test broken Story
   }),
 
   // creates a manifest.json mapping of name to hashed output used in template files
@@ -336,6 +337,7 @@ const config = {
       fs: false,
       vm: require.resolve('vm-browserify'),
       path: false,
+      ...(isDevMode ? { buffer: require.resolve('buffer/') } : {}), // Fix legacy-plugin-chart-paired-t-test broken Story
     },
   },
   context: APP_DIR, // to automatically find tsconfig.json