-
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
[AIRFLOW-3923] Update flask-admin dependency to 1.5.3 #4739
[AIRFLOW-3923] Update flask-admin dependency to 1.5.3 #4739
Conversation
…ity vulnerabilities from safety
Codecov Report
@@ Coverage Diff @@
## master #4739 +/- ##
=======================================
Coverage 74.61% 74.61%
=======================================
Files 430 430
Lines 28002 28002
=======================================
Hits 20893 20893
Misses 7109 7109 Continue to review full report at Codecov.
|
Actually given we remove the old UI which should be the sole place that used flask-admin, do we still need this dependency? |
flask-admin is still referenced in a few areas but I'm not sure if those components are still necessary given the UI rewrite: AirflowViewPlugin - not referenced anywhere in project: https://github.com/apache/airflow/blob/master/airflow/__init__.py#L75 Hive Metastore Browser: https://github.com/apache/airflow/blame/master/airflow/contrib/plugins/metastore_browser/main.py#L48 unit test for plugins, uses BaseView and Menu Link: https://github.com/apache/airflow/blob/master/tests/plugins/test_plugin.py#L101 |
I see. Let's merge your change and then remove the dependency later. |
@feng-tao Until we are done with releases on 1.10 we need to keep up to date there anyway (though perhaps on the v1-10-stable branch for things like this in the future) |
…rity vulnerabilities from safety (#4739)
…rity vulnerabilities from safety (apache#4739)
…rity vulnerabilities from safety (apache#4739)
Jira
Description
36746
36437
Release notes for 1.5.3:
-- sort on multiple columns with column_default_sort
-- sort on related models in column_sortable_list
-- fix: inline model forms can now also be used for models with multiple primary keys
-- support for using mapped column_property
-- handle special characters in filename
-- fix a bug with listing directories on Windows
-- avoid raising an exception when unknown sort parameter is encountered
-- WTForms 3 support
Tests
Airflow usage of flask-admin does not appear to touch any of the changes introduced by 1.5.3
Commits
Documentation
Code Quality
flake8