Closed (fixed)
Project:
Cloud
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
24 Feb 2020 at 12:28 UTC
Updated:
11 Mar 2020 at 10:14 UTC
Jump to comment: Most recent, Most recent file
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 3115567-7.patch | 21.59 KB | yoftahe.addweb |
Comments
Comment #2
yoftahe.addweb commentedComment #3
yas@nileshaddweb
Thank you for the refactoring. Could you please delete the existing view?
Since we replace from
views.view.k8s_deployment_all.ymltoviews.view.k8s_deployment.ymlby add the all view toviews.view.k8s_deployment.ymland by removingviews.view.k8s_deployment_all.yml, the following code needs to work.The URL should remain
/clouds/k8s/deployment.Comment #4
yasComment #5
yoftahe.addweb commented@yas
I did above changes. Could you please review the patch.
Thanks
Comment #6
yas@nileshaddweb
After merging #3115573, it broke this patch. Could you re-create it?
Comment #7
yoftahe.addweb commented@yas
Thanks for review the patch. Please review the recreated patch.
Comment #8
yas@nileshaddweb
Thank you for the update. I tested it and looks good to me. I'll merge the patch to
8.x-1.xand8.x-2.xand close this issue asFixed.Comment #11
yas