Hi Olga, I reviewed your code and the patch is working fine. Assigned the ticket to Brian for the final review. Looks like you already covered all the areas for “edit panel” fields validation, in case if you find any field needs validation, please file the bug for the same. Good Work!!
Thanks and Regards, Sanjay Tripathi From: Olga Smola [mailto:nore...@reviews.apache.org] On Behalf Of Olga Smola Sent: Friday, June 29, 2012 2:34 PM To: Sanjay Tripathi Cc: cloudstack; Olga Smola Subject: Review Request: CS-15391: Missing required fields velidation when Edit on the Details tab for Infrastructure/Zones, Infrastructure/Pods. This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5657/ Review request for cloudstack and Sanjay Tripathi. By Olga Smola. Description Hi Sanjay, Could you please review the ticket http://bugs.cloudstack.org/browse/CS-15391 It's similar to the http://bugs.cloudstack.org/browse/CS-15287 I've assign this ticket to you, because Brian Federle said, that you were working on some validation issues previously. Thanks, Olga Bugs: CS-15391 Diffs * ui/scripts/system.js (6d3b765) View Diff<https://reviews.apache.org/r/5657/diff/>
