----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8992/#review15734 -----------------------------------------------------------
Ship it! Isaac , Thanks for the updated patch . Now , the code changes looks good and works perfectly . I have tested it on my set up as well . I'll merge your changes with asf/master. - Pranav Saxena On Jan. 25, 2013, 9:33 a.m., Isaac Chiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8992/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2013, 9:33 a.m.) > > > Review request for cloudstack and Pranav Saxena. > > > Description > ------- > > The patch is to resolve CLOUDSTACK-965 issue which prevents from showing the > error message and dialog form in the mean time. > > > This addresses bug CLOUDSTACK-965. > > > Diffs > ----- > > ui/scripts/ui/dialog.js 5236bb6 > > Diff: https://reviews.apache.org/r/8992/diff/ > > > Testing > ------- > > > Thanks, > > Isaac Chiang > >
