WIP: Add some helpful comments

lifecycle/source_ldap3.py

@@ -120,7 +120,7 @@ def fetch_groups(self):
                     else:
                         description = ""
                     group = Group(name, description, ldap_group["mail"])
-
+                    # member is of the format uid=admin,cn=users,cn=accounts,...
                     for member in ldap_group["member"]:
                         components = member.split(",")
                         uid = components[0].split("=")[1]

lifecycle/target_ldap3.py

@@ -34,6 +34,11 @@ def _user_to_ldap_changes(user: User) -> dict:
         # 'uid' is also a field that could go into ldap changes, but we don't expect that to ever change
 
         # XXX: HOW DO I SET WHICH USERS ARE MEMBERS OF GROUPS?
+        #      Get the Groups that the User is a part of,
+        #      Search for them in LDAP
+        #      And update every Group entry to add/remove/update the list of "member"
+        #      to include the User's dn.
+        # XXX: Hang on, do users' DN start with "uid=" or "cn="?
         lock_status = "TRUE" if user.locked else "FALSE"
         return {
             "givenName": [(ldap3.MODIFY_REPLACE, [user.forename])],