----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15572/#review29165 -----------------------------------------------------------
Ship it! Ship It! - Ian Duffy On Nov. 15, 2013, 11:50 a.m., Rajani Karuturi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15572/ > ----------------------------------------------------------- > > (Updated Nov. 15, 2013, 11:50 a.m.) > > > Review request for cloudstack, Abhinandan Prateek and Ian Duffy. > > > Bugs: CLOUDSTACK-4866 > https://issues.apache.org/jira/browse/CLOUDSTACK-4866 > > > Repository: cloudstack-git > > > Description > ------- > > added LDAP group name label in add account wizard > > changed the parameter for domain in api importLdapUser from name to UUID > > improved error handling > > > Diffs > ----- > > client/WEB-INF/classes/resources/messages.properties 5885bd0 > > plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapImportUsersCmd.java > 063db0e > > plugins/user-authenticators/ldap/test/groovy/org/apache/cloudstack/ldap/LdapImportUsersCmdSpec.groovy > 0455640 > ui/dictionary.jsp 0ccfc23 > ui/scripts/accountsWizard.js 70ef082 > ui/scripts/docs.js a3151b1 > ui/scripts/ui-custom/accountsWizard.js 358e29c > > Diff: https://reviews.apache.org/r/15572/diff/ > > > Testing > ------- > > unit testing for all the java changes and manual testing for UI changes is > done. > > > Thanks, > > Rajani Karuturi > >
