shubhamraj-git commented on PR #45270: URL: https://github.com/apache/airflow/pull/45270#issuecomment-2565833235
Thanks @jscheffl for working on this. Had the first look. Its going good. As mentioned in TODO, we will support the accordion and not flat. Some points which i noticed, just writing it down incase we don't miss. 1. There is no live changes to conf json onBlur 2. Should have a validation for integer inputs. 3. Dropdown not working for select values 4. we should have a time picker. 5. Length check not working 6. JSON inputs should have a code view? (Just a opinion) 7. List should have multi lines Overall this is going in right direction. Should we have inbuilt scroll for the dynamic form? since the modal seems very long. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
