Merge pull request #872 from GeorgianaElena/uoft-deployer

Consider the case when there is no staff id list
2i2c-org · Dec 6, 2021 · bff4f68 · bff4f68
2 parents 0ed463c + 4566d52
commit bff4f68
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/deployer/utils.py b/deployer/utils.py
@@ -66,7 +66,10 @@ def replace_staff_placeholder(user_list, staff):
         if staff_placeholder in user_list:
             custom_users.remove(staff_placeholder)
 
-            return custom_users + staff_ids
+            custom_users = custom_users + staff_ids
+
+    # If no staff placeholder was provided, then return the original list of users
+    return custom_users
 
 def update_authenticator_config(config, template):
     """Prepare a hub's configuration file for deployment."""