jscheffl commented on PR #55642: URL: https://github.com/apache/airflow/pull/55642#issuecomment-3369097538
@pierrejeambrun thanks for the hints! I attempted to apply your proposals to Edge Executor UI Plugin and it seems to work.... but also needed to realize that https://github.com/apache/airflow/pull/55642/files#diff-72d1ca6b75418dccf2e70084d6d3b9840da702fa127e3848f7fd66198f9d406eR68 was missing on the PR to main. Unfortunately w/o this it is still not working with main. As proposed in https://apache-airflow.slack.com/archives/C0809U4S1Q9/p1759237779941759?thread_ts=1759181599.821269&cid=C0809U4S1Q9 I implemented a detection in "InternalLink" component which captures the diff. ...unfortunately I assume due to missing property in `airflow-core/src/airflow/ui/src/router.tsx` the changes break the functionality in 3.1.0, if I start via `breeze start-airflow --python 3.12 --load-example-dags --backend postgres --executor EdgeExecutor --answer y --use-airflow-version 3.1.0 --use-distributions-from-dist` then I get the following error navigating into plugin. So the changes fix the link in 3.1.1/3.2++ but it breaks with 3.1.0 :-( <img width="1096" height="750" alt="image" src="https://github.com/user-attachments/assets/8aa9ef49-8f2c-400d-966d-ff92ca54eda2"; /> Any idea how I can model a workaround keeping compatibility with 3.1.0 and later versions? -- 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]
