[GitHub] [airflow] dstandish commented on a diff in pull request #31301: Allow optional defaults in required fields with manual triggered dags

["dstandish (via GitHub)" <gi...@apache.org>]

dstandish commented on code in PR #31301:
URL: https://github.com/apache/airflow/pull/31301#discussion_r1336533483


##########
docs/apache-airflow/core-concepts/params.rst:
##########
@@ -309,6 +313,10 @@ The following features are supported in the Trigger UI Form:
   If you want to change values manually, the JSON configuration can be adjusted. Changes are overridden when form fields change.
 - If you want to render custom HTML as form on top of the provided features, you can use the ``custom_html_form`` attribute.
 
+.. note::
+    If the field is required you need to specify a valid default value as well. Exception is if DAG is defined with
+    ``schedule=None`` which enforces the user to provide a value in the form at time of submission.

Review Comment:
   ```suggestion
       ``schedule=None`` which forces the user to provide a value in the form at time of submission.
   ```



-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org