Skip to content

AIP-68 Add loading validation to React App and External Views #52972

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

pierrejeambrun
Copy link
Member

@pierrejeambrun pierrejeambrun commented Jul 7, 2025

Example of conflicting views / apps within the same plugi:n
Screenshot 2025-07-07 at 11 33 58

@pierrejeambrun pierrejeambrun added the log exception Set the label if you want to accept change with caplog label Jul 7, 2025
@pierrejeambrun
Copy link
Member Author

pierrejeambrun commented Jul 7, 2025

Setting log exception label because this is the same pattern of other tests in the very same file.

Copy link
Contributor

@bugraoz93 bugraoz93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Small comment related to caplog which failing on the CI

@pierrejeambrun pierrejeambrun merged commit 0a047bd into apache:main Jul 9, 2025
165 of 167 checks passed
@pierrejeambrun pierrejeambrun deleted the aip-68-add-loading-validation-to-ui-plugins branch July 9, 2025 14:37
HsiuChuanHsu pushed a commit to HsiuChuanHsu/airflow that referenced this pull request Jul 10, 2025
…#52972)

* AIP-68 Add loading validation to React App and External Views

* Small adjustment
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
…#52972)

* AIP-68 Add loading validation to React App and External Views

* Small adjustment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API area:plugins log exception Set the label if you want to accept change with caplog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants