ashb opened a new pull request #18462:
URL: https://github.com/apache/airflow/pull/18462


   The flash is designed for setting a message on one page and then showing
   it after a redirect, so in the case of these UI warnings they were being
   shown twice due to an early redirect.
   
   I could have chosen to fix this by moving the checks to after the
   `reset_tags` redirect, but it _also_ felt wrong to me to have HTML in
   the view, so I have chosen to move it in to the template where it
   belongs.
   
   To (marginally) reduce boilerplate I have created `message()` "macro"
   (Jinja macro, a.k.a. function; not to be confused with what Airflow
   templates call a macro, but is in fact just a template global) that
   handles the formatting for messages.
   
   Closes https://github.com/apache/airflow/issues/17727


-- 
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]


Reply via email to