Skip to content

fix: setupPlugin in chart list page#16413

Merged
graceguo-supercat merged 5 commits into
apache:masterfrom
graceguo-supercat:gg-FixVizTypeDropdown
Aug 25, 2021
Merged

fix: setupPlugin in chart list page#16413
graceguo-supercat merged 5 commits into
apache:masterfrom
graceguo-supercat:gg-FixVizTypeDropdown

Conversation

@graceguo-supercat
Copy link
Copy Markdown

@graceguo-supercat graceguo-supercat commented Aug 23, 2021

SUMMARY

This PR is to fix issue found after #16285. See this comment #16285 (comment): the viz_type dropdown list became empty.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

after:
Screen Shot 2021-08-23 at 3 19 23 PM

TESTING INSTRUCTIONS

CI and manual test.

ADDITIONAL INFORMATION

  • Has associated issue: refactor: re-arrange dashboard page js bundles #16285
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@graceguo-supercat graceguo-supercat force-pushed the gg-FixVizTypeDropdown branch 2 times, most recently from a25765c to 805b853 Compare August 23, 2021 22:53
Copy link
Copy Markdown
Member

@etr2460 etr2460 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense to me

@graceguo-supercat
Copy link
Copy Markdown
Author

graceguo-supercat commented Aug 24, 2021

@ktmud and @villebro, currently CI failed because of this error:

2021-08-24T00:39:47.9629614Z FAIL src/views/CRUD/chart/ChartList.test.jsx
2021-08-24T00:39:47.9630991Z   ● Test suite failed to run
2021-08-24T00:39:47.9631426Z 
2021-08-24T00:39:47.9637059Z     Cannot find module 'file-loader!./countries/austria.geojson' from 'node_modules/@superset-ui/legacy-plugin-chart-country-map/lib/countries.js'
2021-08-24T00:39:47.9638023Z 
2021-08-24T00:39:47.9638460Z     Require stack:
2021-08-24T00:39:47.9639808Z       node_modules/@superset-ui/legacy-plugin-chart-country-map/lib/countries.js
2021-08-24T00:39:47.9644421Z       node_modules/@superset-ui/legacy-plugin-chart-country-map/lib/controlPanel.js
2021-08-24T00:39:47.9647421Z       node_modules/@superset-ui/legacy-plugin-chart-country-map/lib/index.js
2021-08-24T00:39:47.9648478Z       src/visualizations/presets/MainPreset.js
2021-08-24T00:39:47.9649144Z       src/setup/setupPlugins.ts
2021-08-24T00:39:47.9650878Z       src/views/CRUD/chart/ChartList.tsx
2021-08-24T00:39:47.9652831Z       src/views/CRUD/chart/ChartList.test.jsx
2021-08-24T00:39:47.9653304Z 
2021-08-24T00:39:47.9656539Z       at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:306:11)
2021-08-24T00:39:47.9661373Z       at Object.<anonymous> (node_modules/@superset-ui/legacy-plugin-chart-country-map/lib/countries.js:6:39)
2021-08-24T00:39:47.9662736Z 
2021-08-24T00:39:47.9665935Z 
2021-08-24T00:39:47.9673061Z Test Suites: 1 failed, 2 skipped, 337 passed, 338 of 340 total

is something wrong with country-map plugin? Or maybe i need to upgrade to a newer version? thanks!

@graceguo-supercat graceguo-supercat merged commit 08b8aa2 into apache:master Aug 25, 2021
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request Aug 26, 2021
* upstream/master: (25 commits)
  chore(ci): bump pylint to 2.10.2 (apache#16463)
  fix: prevent page crash when chart can't render (apache#16464)
  chore: fixed slack invite link (apache#16466)
  fix(native-filters): handle null values in value filter (apache#16460)
  feat: add function list to auto-complete to Clickhouse datasource (apache#16234)
  refactor(explore): improve typing for Dnd controls (apache#16362)
  fix(explore): update overwrite button on perm change (apache#16437)
  feat: Draggable and Resizable Modal (apache#16394)
  refactor: sql_json view endpoint (apache#16441)
  fix(dashboard): undo and redo buttons weird alignment  (apache#16417)
  fix: setupPlugin in chart list page (apache#16413)
  fix: Disable Slack notification method if no api token (apache#16367)
  feat: add Shillelagh DB engine spec (apache#16416)
  fix: copy to Clipboard order (apache#16299)
  docs: make FEATURE_FLAGS.md reference a link (apache#16415)
  chore(viz): bump superset-ui to 0.17.87 (apache#16420)
  feat: add activate command (apache#16404)
  Revert "fix(explore): let admin overwrite slice (apache#16290)" (apache#16408)
  fix(explore): retain chart ownership on query context update (apache#16419)
  chore: Removes the TODOs and uses the default page size (apache#16422)
  ...
stevenuray pushed a commit to preset-io/superset that referenced this pull request Aug 26, 2021
* fix: setupPlugin in chart list page

* fix the order of setupPlugins call

* Fix jest test on loading geojson

* add jest changes

* fix unit tests

Co-authored-by: Jesse Yang <jesse.yang@airbnb.com>
(cherry picked from commit 08b8aa2)
opus-42 pushed a commit to opus-42/incubator-superset that referenced this pull request Nov 14, 2021
* fix: setupPlugin in chart list page

* fix the order of setupPlugins call

* Fix jest test on loading geojson

* add jest changes

* fix unit tests

Co-authored-by: Jesse Yang <jesse.yang@airbnb.com>
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 28, 2021
* fix: setupPlugin in chart list page

* fix the order of setupPlugins call

* Fix jest test on loading geojson

* add jest changes

* fix unit tests

Co-authored-by: Jesse Yang <jesse.yang@airbnb.com>
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.4.0 First shipped in 1.4.0 labels Mar 13, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
* fix: setupPlugin in chart list page

* fix the order of setupPlugins call

* Fix jest test on loading geojson

* add jest changes

* fix unit tests

Co-authored-by: Jesse Yang <jesse.yang@airbnb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/S 🚢 1.4.0 First shipped in 1.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants