jens-scheffler-bosch commented on issue #31930: URL: https://github.com/apache/airflow/issues/31930#issuecomment-1595349319
Thanks for reporting this bug - and as Part 1 is already fixed with a PR let me answer on the others. Regarding Part 2) Empty strings are converted to `null` values as parameters and if you define a field to be string it is by JSON Schema validation a required field. If you want to have it optional you need to define it as `'param1': Param(default='', type=['null', 'string'])` Regarding Part 3) The implementation of the user form was made in mind that the form fields generate the actual JSON dict which is used to trigger the DAG. It was included in the form because it was just a dict before and for the case you want to manually inspect or tune what is being submitted (or like in your case if you find a problem that you are not blocked). For power user it is a way that you can copy&paste a previous JSON you might have captured from the past. It was not intended to be a 2-way sync back if you modify the JSON back to form fields. This might be a bit more complex to be implemented. I feared (when I contributed the code) this might be a bit over-kill and additional complexity generates more bugs. But if you like to contribute and feel this would be a good feature please feel free to open a PR to contribute. Therefore I assume the bug is resolved, if you have a different opinion, please re-open. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
