-
Notifications
You must be signed in to change notification settings - Fork 443
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
[v1alpha2] New UI #443
Comments
As we discussed today, after we will merge this PR: #438, we should start to work on v1alpha2 version of that New UI. |
i was suggesting to directly make changes for v1alpha2 once grpc apis are merged(this week or early next week). UI has dependency only with APIs and they can even be tested with mock data. I feel that it might be a waste of effort by fixing for v1alpha1 and then again making new changes to v1alpha2 again. |
@johnugeorge Alright |
Any update on this issue? |
@jlewi We have finished UI for v1alpha1 version, we just need to use new API for v1alpha2 and it will work. |
@andreyvelich The PR can be merged as soon as it is ready. It will be released as part of 0.6. |
@andreyvelich What is the remaining work to close out this issue? Are we waiting on #486 to be merged? Do we need to create a separate issue to update the kustomize manifests to use the new UI? |
@jlewi Add comment here: #486 (comment). |
/assign |
Did #486 solve the issue? |
Yes. We can close this issue. I will test our new UI. |
@andreyvelich: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
No description provided.
The text was updated successfully, but these errors were encountered: