----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13959/#review25872 -----------------------------------------------------------
ui/scripts/instances.js <https://reviews.apache.org/r/13959/#comment50444> I think this white spaces might cause problem - Rajesh Battala On Sept. 4, 2013, 2:51 a.m., bo wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13959/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2013, 2:51 a.m.) > > > Review request for cloudstack, Frank Zhang, Jessica Wang, and Min Chen. > > > Bugs: CLOUDSTACK-2605 > > > Repository: cloudstack-git > > > Description > ------- > > It's necessary to hide the button "Add Network to VM Command" for VMs > belonging to Basic Zone,but the public share js file "detailView.js" doesn't > support this,so,I have to change it. > > > Diffs > ----- > > ui/scripts/instances.js 20d3d9a > ui/scripts/ui/widgets/detailView.js 3a59d41 > > Diff: https://reviews.apache.org/r/13959/diff/ > > > Testing > ------- > > This function has worked correctly,and this change doesn't affect other > functions. > > > Thanks, > > bo wang > >