Skip to content

feat: Adds Sunburst chart migration logic#25343

Merged
michael-s-molina merged 2 commits into
apache:masterfrom
michael-s-molina:migrate-sunburst
Sep 22, 2023
Merged

feat: Adds Sunburst chart migration logic#25343
michael-s-molina merged 2 commits into
apache:masterfrom
michael-s-molina:migrate-sunburst

Conversation

@michael-s-molina
Copy link
Copy Markdown
Member

SUMMARY

This PR adds the Sunburst chart migration logic (D3 ➡️ ECharts). Users can execute this migration using the #25304 CLI command and disable the legacy version with the VIZ_TYPE_DENYLIST configuration.

@sadpandajoe @jinghua-qa

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screenshot 2023-09-19 at 14 47 47 Screenshot 2023-09-19 at 14 48 30

TESTING INSTRUCTIONS

1 - Upgrade a Sunburst chart using the CLI command
2 - Check the new chart
3 - Downgrade a Sunburst chart using the CLI command
4 - Check the legacy chart

ADDITIONAL INFORMATION

  • Has associated issue:
  • 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

@justinpark justinpark self-requested a review September 21, 2023 16:48
@michael-s-molina michael-s-molina merged commit 0c083bd into apache:master Sep 22, 2023
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 3.1.0 First shipped in 3.1.0 labels Mar 8, 2024
vinothkumar66 pushed a commit to vinothkumar66/superset that referenced this pull request Nov 11, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
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 🚢 3.1.0 First shipped in 3.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants