api: update_user_group_members should have a user_group_id parameter.

This avoids weirdly passing an extra field in the request, and also
makes the bindings better align with the API.

Author: akashaviator

zulip/zulip/__init__.py

@@ -1454,18 +1454,18 @@ def remove_user_group(self, group_id: int) -> Dict[str, Any]:
             method='DELETE',
         )
 
-    def update_user_group_members(self, group_data: Dict[str, Any]) -> Dict[str, Any]:
+    def update_user_group_members(self, user_group_id: int, group_data: Dict[str, Any]) -> Dict[str, Any]:
         '''
             Example usage:
 
-            >>> client.update_user_group_members({
-                'delete': [4, 8, 15],
-                'add': [16, 23, 42],
+            >>> client.update_user_group_members(1, {
+                'delete': [8, 10],
+                'add': [11],
             })
             {'msg': '', 'result': 'success'}
         '''
         return self.call_endpoint(
-            url='user_groups/{}/members'.format(group_data['group_id']),
+            url='user_groups/{}/members'.format(user_group_id),
             method='POST',
             request=group_data,
         )