zakird / pyad

Python Active Directory Tools | *Not actively maintained*
http://zakird.github.io/pyad/
176 stars 72 forks source link

fixed bug where manager is not defined #131

Closed pwelty1 closed 4 years ago

pwelty1 commented 4 years ago

Hey there!

I am pretty new to making pull requests to public repos. However for my work we built an API using pyad and a website that can manage groups and other AD objects. I was playing with some of the pyad functions and found this error trying to use the ADObject.set_managedby(ADuser) function:

Screenshot 2020-09-29 124836

I did not see anywhere else where manager is not defined. I am assuming we want to assert that the object passed into the function is an ADUser. let me know what you think!