-
Notifications
You must be signed in to change notification settings - Fork 14.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: partitionchart controls migration #9768
feat: partitionchart controls migration #9768
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9768 +/- ##
==========================================
- Coverage 70.89% 64.26% -6.63%
==========================================
Files 585 531 -54
Lines 30427 29079 -1348
Branches 3152 2805 -347
==========================================
- Hits 21572 18689 -2883
- Misses 8744 10212 +1468
- Partials 111 178 +67
Continue to review full report at Codecov.
|
df7c764
to
6606938
Compare
superset-frontend/spec/javascripts/components/ColumnOption_spec.jsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
superset-frontend/spec/javascripts/components/ColumnOption_spec.jsx
Outdated
Show resolved
Hide resolved
* fix lint error * fix: import lint error * pull in latests code * delete control from plugin registry * fixing again... * another fix with rebase... * commit change * update packages and fix test * fix: add ignore to lint error * fix mr * fix lint * fix imports Co-authored-by: Phillip Kelley-Dotson <pkd@pkd.lan>
CATEGORY
Choose one
SUMMARY
Remove partition chart and chart dependencies from incubator to superset.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION
REVIEWERS