-
Notifications
You must be signed in to change notification settings - Fork 142
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 new tab EntityPage for the QuarkusConsole page #1147
Comments
When is the triage scheduled ? |
Apparently, it is only needed to add the following code to the
Remark: I will submit a PR |
Sorry, I didn't see this issue previously. Let me repeat here what I commented on the PR: With dynamic plugins, we should not require changing the source code of the frontend application in order to wire a dynamic plugin in a new tab dedicated to this dynamic plugin. That seems like an anti-pattern. Users of the showcase who need to bring their own external plugins in a dedicated tab will usually not be able to change the showcase container image. That's the whole point of dynamic plugins: always use the same container image without requiring any change to the application code. @tumido @gashcrumb Any idea about how to contribute a new tab ? I assume we should have something, right ? |
See my comment here please: The way @cmoulliard contributes a new tab is totally ok, however there should be someone else driving this decision (IMO)... |
FTR let me post here as well the answer I did to the @tumido comment above:
I think that even this assertion should be validated by PM / UX. |
I also understood our current tab approach as an intentional decision. There's a limit to how many tabs should be used in the tab component and it doesn't currently deal well with lots of tabs. Making this list dynamic is certainly possible, but really needs to be done after working out a strategy to deal with more complicated navigation structures for entity pages. |
Nevertheless this is important for RHDH & Red Hat to include a Runtime tab (aka what Tanzu TAP also proposes) for our customers. Alternatively (if this is doable) we could also decide to merge what we would like to do with the Quarkus/Runtime tab and the existing entity Overview tab |
If that's what you mean, this is already possible to add your cards of you Quarkus tab in the Overview tab. |
#1173 will add the ability to configure new tabs as needed. You'll be able to add new sub-paths to catalog entity pages and define the mount points that will be rendered at those sub-paths. |
Provide a clear and concise description of the feature request
We cannot until now use our QuarkusConsole page
as the
EntityPage
on RHDH/showcase only supports the following mountPoints and no tab exists here for QuarkusThe text was updated successfully, but these errors were encountered: