The code change looks sensible, but a couple of things: 1. We should add a test that covers this - probably in tests/www/test_views.py as a new test function in `TestVariableView` 2. Does this fix the form or is there a validation error saying value is required? 3. (Annoyingly, right now) there are two WWW versions that need updating. There is also www_rbac (which will become the only version, but for now please update both versions and both tests)
Thanks! [ Full content available at: https://github.com/apache/incubator-airflow/pull/3948 ] This message was relayed via gitbox.apache.org for [email protected]
