sjyangkevin commented on code in PR #74528:
URL: https://github.com/apache/airflow/pull/74528#discussion_r4253562624


##########
providers/standard/src/airflow/providers/standard/sensors/external_task.py:
##########
@@ -65,12 +67,22 @@
 
     from airflow.providers.common.compat.sdk import Context, TaskInstanceKey
 
+XCOM_EXTERNAL_RUN_IDS = "external_run_ids"
+
+
+def _build_airflow_ui_url(path: str) -> str:
+    base_url = conf.get("api", "base_url", fallback="/")
+    return urljoin(base_url.rstrip("/") + "/", path)
+
 
 class ExternalDagLink(BaseOperatorLink):
     """
     Operator link for ExternalTaskSensor and ExternalTaskMarker.
 
     It allows users to access DAG waited with ExternalTaskSensor or cleared by 
ExternalTaskMarker.
+
+    On Airflow 3, the link opens the external Dag run that ExternalTaskSensor 
matched, the external
+    Dag's runs list if it matched several runs, or the external Dag itself if 
no run is known yet.

Review Comment:
   thanks for the feedback. the comments had been removed.



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