Skip to content

Commit 96ef26c

Browse files
committed
Correct string conversion and coroutine run
1 parent 613e6a9 commit 96ef26c

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

arthur/exts/directory/ldap.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -198,9 +198,11 @@ async def sync_users(self) -> None:
198198

199199
for user in diff:
200200
await self._process_user(user, notified_users)
201-
handled.add(user.discord_user.id)
201+
handled.add(
202+
user.ldap_user.employee_number if user.ldap_user else str(user.discord_user.id)
203+
)
202204

203-
self._handle_left_users(handled)
205+
await self._handle_left_users(handled)
204206

205207
logger.info("LDAP: Sync complete.")
206208
except Exception as e: # noqa: BLE001
@@ -209,9 +211,9 @@ async def sync_users(self) -> None:
209211
f":x: LDAP Sync Error: ```python\n{e}```"
210212
)
211213

212-
def _handle_left_users(self, handled: list[int]) -> None:
214+
async def _handle_left_users(self, handled: list[int]) -> None:
213215
"""Handle users that have left the guild and so were not processed."""
214-
ldap_users = ldap.find_users()
216+
ldap_users = await ldap.find_users()
215217

216218
for user in ldap_users:
217219
if user.employee_number is None or user.employee_number in handled:

0 commit comments

Comments
 (0)