-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
fix: clean up chart metadata config #19143
Conversation
superset-frontend/packages/superset-ui-core/src/chart/models/ChartMetadata.ts
Show resolved
Hide resolved
Thank you for the fixes here! |
Codecov Report
@@ Coverage Diff @@
## master #19143 +/- ##
==========================================
- Coverage 66.55% 66.55% -0.01%
==========================================
Files 1646 1646
Lines 63617 63625 +8
Branches 6471 6475 +4
==========================================
+ Hits 42340 42344 +4
- Misses 19599 19602 +3
- Partials 1678 1679 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
🏷️ preset:2022.11 |
SUMMARY
Removing the
deprecated
field turns out to have been a bit much. Separately, flattening the label object into two separate fields allows us to better specify defaults and overrides for things.BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
tested both deprecation and labels, and they still work as before.
TESTING INSTRUCTIONS
Set a plugin's metadata to have
deprecated: true
orlabel: ChartLabel.FEATURED
and find it (or don't, if it's deprecated) in the viz picker.ADDITIONAL INFORMATION