-
Notifications
You must be signed in to change notification settings - Fork 15.2k
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
base: main
Are you sure you want to change the base?
Adjustment external views #51889
Conversation
Second commit looks weird, (pre-commit hook formatting for generated files). Fixed in: #51890 |
4c79e8e
to
74dfc89
Compare
There was a problem hiding this 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done thanks.
74dfc89
to
06d88e6
Compare
Closes: #51855