kaxil commented on code in PR #50197:
URL: https://github.com/apache/airflow/pull/50197#discussion_r2075504808


##########
airflow-core/docs/howto/define-extra-link.rst:
##########
@@ -62,6 +62,9 @@ The following code shows how to add extra links to an 
operator via Plugins:
 
 .. note:: Operator Extra Links should be registered via Airflow Plugins or 
custom Airflow Provider to work.

Review Comment:
   Shouldn't it work from any place? The initial reason we had restricted it 
from Airflow plugins was because it was loading user code in the webserver 
during the deserialization process. But now that we store those in xcoms and UI 
reads it from there, what's restricts this to Plugins?



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