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

refactor: move chart to core #760

Merged
merged 3 commits into from
Sep 3, 2020

Conversation

ktmud
Copy link
Contributor

@ktmud ktmud commented Aug 27, 2020

💔 Breaking Changes

🏆 Enhancements

The final and biggest PR for merging superset-ui packages.

Related: #728 #729 #730 #732 #755 #756 #757 #759

@ktmud ktmud requested a review from a team as a code owner August 27, 2020 21:04
@vercel
Copy link

vercel bot commented Aug 27, 2020

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

🔍 Inspect: https://vercel.com/superset/superset-ui/qfkiy8v0b
✅ Preview: https://superset-ui-git-fork-ktmud-monopackage-move-chart.superset.vercel.app

@codecov
Copy link

codecov bot commented Aug 31, 2020

Codecov Report

Merging #760 into monopackage will increase coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           monopackage     #760      +/-   ##
===============================================
+ Coverage        24.79%   24.90%   +0.10%     
===============================================
  Files              352      353       +1     
  Lines             7845     7839       -6     
  Branches          1003     1003              
===============================================
+ Hits              1945     1952       +7     
+ Misses            5796     5785      -11     
+ Partials           104      102       -2     
Impacted Files Coverage Δ
...ntrols/src/components/CertifiedIconWithTooltip.tsx 33.33% <ø> (ø)
...ckages/superset-ui-chart-controls/src/constants.ts 100.00% <ø> (ø)
...i-chart-controls/src/utils/expandControlConfig.tsx 100.00% <ø> (+26.66%) ⬆️
.../superset-ui-core/src/chart/clients/ChartClient.ts 100.00% <ø> (ø)
...ui-core/src/chart/components/ChartDataProvider.tsx 100.00% <ø> (ø)
...ui-core/src/chart/components/FallbackComponent.tsx 100.00% <ø> (ø)
...i-core/src/chart/components/NoResultsComponent.tsx 100.00% <ø> (ø)
...perset-ui-core/src/chart/components/SuperChart.tsx 100.00% <ø> (ø)
...et-ui-core/src/chart/components/SuperChartCore.tsx 100.00% <ø> (ø)
...ore/src/chart/components/createLoadableRenderer.ts 100.00% <ø> (ø)
... and 99 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 391f4cd...4ec0ca3. Read the comment docs.

@ktmud ktmud force-pushed the monopackage branch 2 times, most recently from 5d86af4 to 391f4cd Compare September 2, 2020 21:45
@ktmud ktmud merged commit 8a4f8d8 into apache-superset:monopackage Sep 3, 2020
@ktmud ktmud deleted the monopackage-move-chart branch September 3, 2020 17:33
@ktmud ktmud changed the title refactor: move chart and chart-controls to core refactor: move chart to core Sep 3, 2020
ktmud added a commit that referenced this pull request Sep 3, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8: move superset-ui/chart-composition to core (#759)
9: move superset-ui/chart to core (#760)
ktmud added a commit that referenced this pull request Sep 3, 2020
1. move translation, connection, query to core (#729)
2. update encodable to remove formatter inter-dependency (#744)
3. move number-format and time-format to core (#730)
4. move superset-ui/dimension to core (#732)
5. move superset-ui/color to core (#755)
6. move superset-ui/style to core (#756)
7. move superset-ui/validator to core (#757)
8. move superset-ui/chart-composition to core (#759)
9. move superset-ui/chart to core (#760)
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.

2 participants