[GitHub] SudharmaJain opened a new pull request #2400: CLOUDSTACK-8900 listLdapUsers with listType=new only filters users for logged-in user account

Wed, 10 Jan 2018 02:16:45 -0800 

SudharmaJain opened a new pull request #2400: CLOUDSTACK-8900 listLdapUsers 
with listType=new only filters users for logged-in user account
URL: https://github.com/apache/cloudstack/pull/2400
 
 
   Added an ldap user 'ldapuser1' to admin account. Now on clicking 'Add LDAP 
Account' button, UI displays the ldap users other than 'ldapuser1'.  
   
   
![image](https://user-images.githubusercontent.com/12229259/34766536-b615f47c-f61a-11e7-9a3c-8e7e0d7ee7ad.png)
   
   
![image](https://user-images.githubusercontent.com/12229259/34766552-c0c5e8e6-f61a-11e7-968d-29cac3a046f1.png)
   
   Added a new admin account as 'admin2'. Login as admin2 and clicking on  'Add 
LDAP Account' displays all the ldap users including 'ldapuser1'. 
   
   ## Expected behavior
   
   List Ldapusers with listtype=new should filter already added users 
irrespective of logged in user.
   
   ## Before the Fix
   
   
![image](https://user-images.githubusercontent.com/12229259/34766571-decf6a1a-f61a-11e7-85d9-2c7cf712600d.png)
   ## After the Fix
   
![image](https://user-images.githubusercontent.com/12229259/34766581-e3293aa0-f61a-11e7-9a51-4828a4fb575c.png)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to