ashb commented on code in PR #52996: URL: https://github.com/apache/airflow/pull/52996#discussion_r2192134826
########## airflow-core/src/airflow/api_fastapi/core_api/datamodels/ui/config.py: ########## @@ -31,6 +31,7 @@ class ConfigResponse(BaseModel): enable_swagger_ui: bool require_confirmation_dag_change: bool default_wrap: bool + default_show_timestamp: bool Review Comment: @bbovenzi @pierrejeambrun I'm a little surprised to see in the server -- do purely UI concerns such as wrap and timestamp even make sense to have here? They aren't configurable via Airlfow config, so these values don't ever change do they? -- 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