-
Notifications
You must be signed in to change notification settings - Fork 16
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
Initial UI refresh for the grafana integration #182
Comments
The following navigation entries would be available post the refresh from the ui-legacy-code branch, which was created as part of the codebase refresh addressed in Tendrl/ui#486.
|
The following functionality would be carried forward from the ui-legacy-code branch, which was created as part of the codebase refresh addressed in Tendrl/ui#486.
Other functionality such as workflows, list views and the various object details views would be ported subsequently. The updated cluster list views are being addressed in spec #181, while the import flow would be ported in spec #183. |
I have several doubts about this issue. As the issue doesn't correspond to what was said in #200 Especially these two points makes me wonder:
As I see it:
So my question is what design is followed, if any? |
@ltrilety, This setup is for Initial milestone, the specification and setup will be updated as soon as designs get finalized and we move towards other milestones. |
@a2batic thanks for answer, it's just I don't understand what this particular changes improve in the long term point of view. If it should be the raw base from which a new UI will be created, I don't know why there is still |
@ltrilety - we are recommending going with Design B, and we're waiting on @sankarshanmukhopadhyay and Team to advise which Milestone it will land in. |
Closing this issue as the PRs associated with this issue are merged |
Cluster specific objects like volumes and pools are to be removed from the main navigation bar. These views would only be available under a specific cluster. Update the navigation to allow quick switching between different clusters.
The text was updated successfully, but these errors were encountered: