Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

feat: migrate chart plugins #375

Merged
merged 12 commits into from
Apr 17, 2020
Merged

feat: migrate chart plugins #375

merged 12 commits into from
Apr 17, 2020

Conversation

kristw
Copy link
Contributor

@kristw kristw commented Apr 17, 2020

🏆 Enhancements

Migrate

  • force directed
  • histogram
  • mapbox
  • rose
  • sankey
  • sankey loop
  • sunburst
  • treemap
  • world map

Most of the changes are eslint auto fix via yarn lint:fix.
And update storybook structure to match the new way.

@kristw kristw requested a review from a team as a code owner April 17, 2020 08:02
@vercel
Copy link

vercel bot commented Apr 17, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/superset/superset-ui/9232jhg77
✅ Preview: https://superset-ui-git-kristw-migrate-more.superset.now.sh

@codecov
Copy link

codecov bot commented Apr 17, 2020

Codecov Report

Merging #375 into master will decrease coverage by 5.19%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
- Coverage   32.03%   26.83%   -5.20%     
==========================================
  Files         155      188      +33     
  Lines        4308     5161     +853     
  Branches      422      459      +37     
==========================================
+ Hits         1380     1385       +5     
- Misses       2899     3747     +848     
  Partials       29       29              
Impacted Files Coverage Δ
...y-plugin-chart-force-directed/src/ForceDirected.js 0.00% <0.00%> (ø)
...ns/legacy-plugin-chart-force-directed/src/index.js 0.00% <ø> (ø)
...-plugin-chart-force-directed/src/transformProps.js 0.00% <ø> (ø)
...ns/legacy-plugin-chart-histogram/src/Histogram.jsx 0.00% <ø> (ø)
...s/legacy-plugin-chart-histogram/src/WithLegend.jsx 0.00% <ø> (ø)
plugins/legacy-plugin-chart-histogram/src/index.js 0.00% <ø> (ø)
...egacy-plugin-chart-histogram/src/transformProps.js 0.00% <ø> (ø)
plugins/legacy-plugin-chart-map-box/src/MapBox.jsx 0.00% <ø> (ø)
...lugin-chart-map-box/src/ScatterPlotGlowOverlay.jsx 0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-map-box/src/index.js 0.00% <ø> (ø)
... and 56 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 106377a...5f7af64. Read the comment docs.

@@ -58,6 +58,75 @@ function getAncestors(node) {
return path;
}

function buildHierarchy(rows) {
Copy link
Contributor Author

@kristw kristw Apr 17, 2020

Choose a reason for hiding this comment

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

Just move function outside to please eslint

Copy link
Contributor

@villebro villebro left a comment

Choose a reason for hiding this comment

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

LGTM. To make reviewing easier, it would be convenient to have a respective removal PR in superset-ui-plugins available to contrast with, just to make sure nothing is lost in transit.

@kristw
Copy link
Contributor Author

kristw commented Apr 17, 2020

Removal PR: apache-superset/superset-ui-plugins#440

Copy link
Contributor

@ktmud ktmud left a comment

Choose a reason for hiding this comment

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

LGTM!

@kristw kristw merged commit 7c8cbd8 into master Apr 17, 2020
@delete-merged-branch delete-merged-branch bot deleted the kristw--migrate-more branch April 17, 2020 22:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants