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

Reply via email to