-
Notifications
You must be signed in to change notification settings - Fork 667
Add ConsoleLink CRD to enable custom links in help and user dropdowns #1360
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
Add ConsoleLink CRD to enable custom links in help and user dropdowns #1360
Conversation
d33c08c to
9705a43
Compare
9705a43 to
dfd03f5
Compare
7c3c1bd to
7ceee4a
Compare
907cbcb to
6c19d19
Compare
6c19d19 to
808b850
Compare
6ceb328 to
f55bb5f
Compare
32c6a9a to
ad7e7c5
Compare
ad7e7c5 to
6dd65ec
Compare
5aee65f to
bf10831
Compare
|
The CRD file can be removed from this PR. |
bf10831 to
76c8e46
Compare
c1a4c1a to
2198e50
Compare
|
/retest |
|
@spadgett we should create extensions for all these areas of contribution and then have the CRD simply populate the extensions |
2198e50 to
8538548
Compare
|
/retest |
8538548 to
44ff079
Compare
Makes sense, I agree. If you don't mind, I suggest we get the initial change in and then make it a proper extension in a follow-on so we don't miss 4.2. This PR has been blocked for about 3 months on API changes and was originally written before static plugins were added! |
spadgett
left a comment
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.
| spec: | ||
| href: 'https://www.example.com' | ||
| location: HelpMenu | ||
| text: Additional help menu link |
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.
nit: We probably want title case in the YAML template since the other menu items are title case
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhamilto, spadgett The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@christianvogt I opened an issue to make this change: https://jira.coreos.com/browse/CONSOLE-1571 |
https://jira.coreos.com/browse/CONSOLE-1483