Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[2.0] change the ENABLE_JAVASCRIPT_CONTROLS from a config value to a feature flag #18909

Closed
suddjian opened this issue Feb 24, 2022 · 2 comments · Fixed by #19113
Closed

[2.0] change the ENABLE_JAVASCRIPT_CONTROLS from a config value to a feature flag #18909

suddjian opened this issue Feb 24, 2022 · 2 comments · Fixed by #19113
Assignees
@rusackas
Labels
need:more-info Requires more information from author preset-io

Comments

@suddjian
Copy link
Member

No description provided.

@request-info
Copy link

request-info bot commented Feb 24, 2022

We would appreciate it if you could provide us with more info about this issue/pr! Please do not leave the title or description empty.

@request-info request-info bot added the need:more-info Requires more information from author label Feb 24, 2022
@suddjian suddjian moved this to Todo in Apache Superset 2.0 Feb 24, 2022
@rusackas rusackas self-assigned this Feb 25, 2022
@rusackas rusackas moved this from Todo to In Progress in Apache Superset 2.0 Mar 2, 2022
@rusackas
Copy link
Member

rusackas commented Mar 3, 2022

There are bugs in this feature, but it's off by default still in 2.0, so we won't concern ourselves here.

@rusackas rusackas mentioned this issue Mar 10, 2022
Merged
8 tasks
Repository owner moved this from In Progress to Done in Apache Superset 2.0 Mar 14, 2022
@suddjian suddjian mentioned this issue Mar 18, 2022
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rusackas rusackas

Labels
need:more-info Requires more information from author preset-io
Projects
No open projects
Apache Superset 2.0
Status: Done
Milestone
No milestone
2 participants
@rusackas @suddjian