Skip to content

Adjustment external views #51889

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pierrejeambrun
Copy link
Member

Closes: #51855

@pierrejeambrun
Copy link
Member Author

pierrejeambrun commented Jun 18, 2025

Second commit looks weird, (pre-commit hook formatting for generated files).

Fixed in: #51890

@pierrejeambrun pierrejeambrun force-pushed the 51855-update-iframe-views branch 2 times, most recently from 4c79e8e to 74dfc89 Compare June 18, 2025 15:33
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.

Overall, it looks great! One test and UI Lint static check are falling in the CI. Might be great to get additional reviews for the UI side :)

# Optional parameters, relative URL location for the iframe rendering. If not provided, external view will be rendeded as an external link. Shouldn
# not contain a leading slash.
"url_route": "my_iframe_view",
# Optional category, only relevant for destination "nav". This is used to group the external links in the navigation bar.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Optional category, only relevant for destination "nav". This is used to group the external links in the navigation bar.
# Optional category, only relevant for destination "nav". This is used to group the external links in the navigation bar. We will match the existing menus of ["browse", "docs", "admin", "user"] and if there's no match then create a new menu

Copy link
Member Author

Choose a reason for hiding this comment

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

Done thanks.

@pierrejeambrun pierrejeambrun force-pushed the 51855-update-iframe-views branch from 74dfc89 to 06d88e6 Compare June 19, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:airflow-ctl area:API Airflow's REST/HTTP API area:CLI area:plugins area:UI Related to UI/UX. For Frontend Developers. kind:documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AIP-68 | Merge iframe_views and appbuilder_menu_items
4 participants