> On Oct. 8, 2014, 11:46 a.m., Rohit Yadav wrote: > > Looks like an additional jquery-validate library was added, can you explain > > what this does such as validates what in the frontend?
It adds extra validation capabilities to the jquery-validation plugin that we are already using. At the moment none of them is used, but once this has been committed I will quickly add validations for ipv4, ipv6, url fields for example. It will be that easy: ip_address_field: { required: true, ipv4: true //this will use the ipv4 validator defined in the additional-methods lib } - Gabor ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26053/#review55783 ----------------------------------------------------------- On Sept. 30, 2014, 5:12 p.m., Gabor Apati-Nagy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26053/ > ----------------------------------------------------------- > > (Updated Sept. 30, 2014, 5:12 p.m.) > > > Review request for cloudstack, Brian Federle and Jessica Wang. > > > Bugs: CLOUDSTACK-7659 > https://issues.apache.org/jira/browse/CLOUDSTACK-7659 > > > Repository: cloudstack-git > > > Description > ------- > > This patch upgrades jquery-validation library and also adds the > additional-methods.js extension that offers additional validation options > (same license applies, same publisher). I also updated the LICENSE file to > show the added file as well. > > > Diffs > ----- > > LICENSE 497323bffce2b2afab28c187cfdafa960944ed94 > ui/index.jsp e0b4b7848175e3f2159d44d6ac48d122858a2854 > ui/lib/jquery.validate.additional-methods.js PRE-CREATION > ui/lib/jquery.validate.js 72296a61f208298f2e54ac7b54b70557f41e0f84 > > Diff: https://reviews.apache.org/r/26053/diff/ > > > Testing > ------- > > Tested that the validation is still working as expected. > > > Thanks, > > Gabor Apati-Nagy > >