Code review comment for lp:~lamont/maas/bug-1660188

Revision history for this message
Blake Rouse (blake-rouse) wrote :

Overall almost there. Couple of issues with the directives, just always use ng-if instead of ng-show or ng-hide. ng-if is better performance as it actually removes the dom elements from the page. There has been some effort to transition all to ng-if but that has not happened 100% yet.

I did run into this issue on the devices details page. The DHCP warning shouldn't be there and I think editing should be possible for a device if a rack controller if no rack controllers. I really want to remove that limitation for machines as well, but not yet, sigh...

review: Needs Fixing

« Back to merge proposal