Merge pull request #7 from d33bs/main

Resolve user request issues for #6
UCBoulder · Mar 7, 2022 · 057e338 · 057e338
2 parents 0ca0237 + b152524
commit 057e338
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/setup.py b/setup.py
@@ -20,7 +20,7 @@
 # This call to setup() does all the work
 setuptools.setup(
     name="zoom_client",
-    version="0.0.4",
+    version="0.0.5",
     description="Zoom (Video Communications) API Client",
     long_description=README,
     long_description_content_type="text/markdown",

diff --git a/zoom_client/modules/users.py b/zoom_client/modules/users.py
@@ -143,7 +143,7 @@ def get_current_users(self):
         @sleep_and_retry
         @limits(calls=60, period=5)
         def make_requests(
-            page_number: int = 1, page_count: int = None, result_list: list = None
+            page_number: int = 1, page_count: int = 0, result_list: list = []
         ) -> list:
 
             logging.info("Making user request %s of %s", page_number, page_count)
@@ -153,7 +153,7 @@ def make_requests(
             )
 
             # if no users are returned in the result, we break our loop
-            if "users" in result:
+            if "users" in result.keys():
                 user_results = result["users"]
                 result_list += user_results