You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/08 20:52:57 UTC

[GitHub] [airflow] kaxil commented on a change in pull request #19453: Improve message and documentation around moved data

kaxil commented on a change in pull request #19453:
URL: https://github.com/apache/airflow/pull/19453#discussion_r745084105



##########
File path: airflow/www/templates/airflow/dags.html
##########
@@ -57,7 +57,8 @@
       Airflow found incompatible data in the <code>{{ original_table_name }}</code> table in the
       metadatabase, and has moved them to <code>{{ moved_table_name }}</code> during the database migration
       to upgrade. Please inspect the moved data to decide whether you need to keep them, and manually drop
-      the <code>{{ moved_table_name }}</code> table to dismiss this warning.
+      the <code>{{ moved_table_name }}</code> table to dismiss this warning.<p/>
+     Read more about it in <a href={{ get_docs_url("installing/upgrading.html") }}><b>Upgrading</b></a>

Review comment:
       https://stackoverflow.com/a/1261157/5691525 -- suggests some alternatives instead of `<p/>`
   
   Maybe @bbovenzi or @ryanahamilton can help here




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