----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12969/#review24105 -----------------------------------------------------------
Ship it! shipped to ldapplugin ec064b307734dcb1d4304454de466d6e70e019fb close as submitted - Sebastien Goasguen On July 26, 2013, 4:13 p.m., Ian Duffy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12969/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 4:13 p.m.) > > > Review request for cloudstack, Abhinandan Prateek, Pranav Saxena, and > Sebastien Goasguen. > > > Repository: cloudstack-git > > > Description > ------- > > Completes the task for adding LDAP user provisioning UI features. > > This inlcudes: > Add accounts -> add account > Table of user accounts within LDAP appears. > Admin selects user and can add. > An admin has the ability to select multiple users. > > This add screen will only appear when ldap is enabled. > > > Diffs > ----- > > plugins/user-authenticators/ldap/test/resources/cloudstack.org.ldif b20de81 > ui/css/cloudstack3.css 93658db > ui/index.jsp 813dbcb > ui/scripts/accounts.js e3e28e4 > ui/scripts/accountsWizard.js a350085 > ui/scripts/sharedFunctions.js 6c1b009 > ui/scripts/ui-custom/accountsWizard.js 4037788 > > Diff: https://reviews.apache.org/r/12969/diff/ > > > Testing > ------- > > All testing was done manually > > Create a single account when ldap is enabled. > create multiple accounts when ldap is enabled. > > create a single account when ldap is disabled. > > > Thanks, > > Ian Duffy > >