rusackas commented on PR #36275: URL: https://github.com/apache/superset/pull/36275#issuecomment-3590360502
Once all controls are actual React componets, we can take on next steps: 1) Remove the string-based control reference parsing code 2) Remove the config object based control reference parsing code 3) Wire up proper statefulness (for dynamic controls) to be handled at the control panel layer (Superset-wide Redux _probably_ needn't care about this) 4) Replace the array-of-arrays control panel layout with JSONForms. Then we can put the Controls in Storybook and developer docs, strip out a ton of code and complexity, have more interactive control panels, create "chunks" of controls as shared macro-components, standardize our control panel layouts, and take over the unvierse! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
