Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-09 Thread via GitHub
pierrejeambrun merged PR #52972: URL: https://github.com/apache/airflow/pull/52972 -- 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-unsubscr...@a

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-09 Thread via GitHub
bugraoz93 commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2195068099 ## airflow-core/tests/unit/plugins/test_plugins_manager.py: ## @@ -151,6 +151,48 @@ class AirflowAdminMenuLinksPlugin(AirflowPlugin): ), ] +

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-09 Thread via GitHub
bugraoz93 commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2195046995 ## airflow-core/tests/unit/plugins/test_plugins_manager.py: ## @@ -151,6 +151,48 @@ class AirflowAdminMenuLinksPlugin(AirflowPlugin): ), ] +

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-08 Thread via GitHub
bbovenzi commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2192847268 ## airflow-core/src/airflow/api_fastapi/app.py: ## @@ -93,6 +94,7 @@ def create_app(apps: str = "all") -> FastAPI: init_plugins(app) init_auth_mana

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-08 Thread via GitHub
pierrejeambrun commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2192249233 ## airflow-core/tests/unit/plugins/test_plugins_manager.py: ## @@ -151,6 +151,48 @@ class AirflowAdminMenuLinksPlugin(AirflowPlugin): ),

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-08 Thread via GitHub
pierrejeambrun commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2192248609 ## airflow-core/tests/unit/plugins/test_plugins_manager.py: ## @@ -151,6 +151,48 @@ class AirflowAdminMenuLinksPlugin(AirflowPlugin): ),

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-07 Thread via GitHub
bugraoz93 commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2190928878 ## airflow-core/tests/unit/plugins/test_plugins_manager.py: ## @@ -151,6 +151,48 @@ class AirflowAdminMenuLinksPlugin(AirflowPlugin): ), ] +

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-07 Thread via GitHub
pierrejeambrun closed pull request #52972: AIP-68 Add loading validation to React App and External Views URL: https://github.com/apache/airflow/pull/52972 -- 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 g

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-07 Thread via GitHub
pierrejeambrun commented on PR #52972: URL: https://github.com/apache/airflow/pull/52972#issuecomment-3045653434 Seting `log exception` label because this is the same pattern of other tests in the very same file. -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-07 Thread via GitHub
pierrejeambrun commented on code in PR #52972: URL: https://github.com/apache/airflow/pull/52972#discussion_r2189491154 ## airflow-core/src/airflow/api_fastapi/app.py: ## @@ -93,6 +94,7 @@ def create_app(apps: str = "all") -> FastAPI: init_plugins(app) init_aut

[PR] AIP-68 Add loading validation to React App and External Views [airflow]

2025-07-07 Thread via GitHub
pierrejeambrun opened a new pull request, #52972: URL: https://github.com/apache/airflow/pull/52972 (no comment) -- 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 unsubscr