-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
[2.0] generally remove all code marked as "deprecated" (front end) #19015
Comments
@eschutho and I went through superset-frontend code base and found 2 deprecation candidates:
But both of them are currently in use and no replacement code available. So we think it's not a right time to deprecate them right now. |
Sweep through and change any comments like "will be deprecated in 2.0" to be "will be deprecated in a future version" |
Consider removing Omnibar for 3.0 - @suddjian we should start 3.0 proposals board |
Given this discussion: #13305 (comment), we should consider remove this feature (now or 3.0) |
I believe that since it was already marked as deprecated in the code, it's fair game to remove as part of this initiative. That work could be part of this Issue, or filed a separate one if need be, but I have no objection to the removal as part of 2.0. |
Look for comments stating that things that will be removed for, and remove them
The text was updated successfully, but these errors were encountered: