Add get_meeting in coll and conn, add bot_image_url param in record meeting, fix repr

Author: ashish-spext

videodb/client.py

@@ -45,7 +45,9 @@ def __init__(self, api_key: str, base_url: str, **kwargs) -> "Connection":
         self.api_key = api_key
         self.base_url = base_url
         self.collection_id = "default"
-        super().__init__(api_key=api_key, base_url=base_url, version=__version__, **kwargs)
+        super().__init__(
+            api_key=api_key, base_url=base_url, version=__version__, **kwargs
+        )
 
     def get_collection(self, collection_id: Optional[str] = "default") -> Collection:
         """Get a collection object by its ID.
@@ -272,8 +274,8 @@ def upload(
         :param str name: Name of the file (optional)
         :param str description: Description of the file (optional)
         :param str callback_url: URL to receive the callback (optional)
-        :param str file_path: Path to the file to upload (optional) 
-        :param str url: URL of the file to upload (optional) 
+        :param str file_path: Path to the file to upload (optional)
+        :param str url: URL of the file to upload (optional)
         :return: :class:`Video <Video>`, or :class:`Audio <Audio>`, or :class:`Image <Image>` object
         :rtype: Union[ :class:`videodb.video.Video`, :class:`videodb.audio.Audio`, :class:`videodb.image.Image`]
         """
@@ -298,16 +300,18 @@ def upload(
     def record_meeting(
         self,
         link: str,
-        bot_name: str,
-        meeting_name: str,
-        callback_url: str,
+        bot_name: str = None,
+        bot_image_url: str = None,
+        meeting_name: str = None,
+        callback_url: str = None,
         callback_data: dict = {},
         time_zone: str = "UTC",
     ) -> Meeting:
         """Record a meeting and upload it to the default collection.
 
         :param str link: Meeting link
         :param str bot_name: Name of the recorder bot
+        :param str bot_image_url: URL of the recorder bot image
         :param str meeting_name: Name of the meeting
         :param str callback_url: URL to receive callback once recording is done
         :param dict callback_data: Data to be sent in the callback (optional)
@@ -321,6 +325,7 @@ def record_meeting(
             data={
                 "link": link,
                 "bot_name": bot_name,
+                "bot_image_url": bot_image_url,
                 "meeting_name": meeting_name,
                 "callback_url": callback_url,
                 "callback_data": callback_data,
@@ -329,3 +334,14 @@ def record_meeting(
         )
         meeting_id = response.get("meeting_id")
         return Meeting(self, id=meeting_id, collection_id="default", **response)
+
+    def get_meeting(self, meeting_id: str) -> Meeting:
+        """Get a meeting by its ID.
+
+        :param str meeting_id: ID of the meeting
+        :return: :class:`Meeting <Meeting>` object
+        :rtype: :class:`videodb.meeting.Meeting`
+        """
+        meeting = Meeting(self, id=meeting_id, collection_id="default")
+        meeting.refresh()
+        return meeting

videodb/collection.py

@@ -517,6 +517,7 @@ def record_meeting(
         self,
         link: str,
         bot_name: str = None,
+        bot_image_url: str = None,
         meeting_name: str = None,
         callback_url: str = None,
         callback_data: dict = {},
@@ -526,6 +527,7 @@ def record_meeting(
 
         :param str link: Meeting link
         :param str bot_name: Name of the recorder bot
+        :param str bot_image_url: URL of the recorder bot image
         :param str meeting_name: Name of the meeting
         :param str callback_url: URL to receive callback once recording is done
         :param dict callback_data: Data to be sent in the callback (optional)
@@ -539,11 +541,25 @@ def record_meeting(
             data={
                 "link": link,
                 "bot_name": bot_name,
+                "bot_image_url": bot_image_url,
                 "meeting_name": meeting_name,
                 "callback_url": callback_url,
                 "callback_data": callback_data,
                 "time_zone": time_zone,
             },
         )
         meeting_id = response.get("meeting_id")
-        return Meeting(self._connection, id=meeting_id, collection_id=self.id, **response)
+        return Meeting(
+            self._connection, id=meeting_id, collection_id=self.id, **response
+        )
+
+    def get_meeting(self, meeting_id: str) -> Meeting:
+        """Get a meeting by its ID.
+
+        :param str meeting_id: ID of the meeting
+        :return: :class:`Meeting <Meeting>` object
+        :rtype: :class:`videodb.meeting.Meeting`
+        """
+        meeting = Meeting(self._connection, id=meeting_id, collection_id=self.id)
+        meeting.refresh()
+        return meeting

videodb/meeting.py

@@ -26,7 +26,7 @@ def __init__(self, _connection, id: str, collection_id: str, **kwargs) -> None:
         self._update_attributes(kwargs)
 
     def __repr__(self) -> str:
-        return f"Meeting(id={self.id}, collection_id={self.collection_id}, name={self.name}, status={self.status}, bot_name={self.bot_name})"
+        return f"Meeting(id={self.id}, collection_id={self.collection_id}, name={self.name}, status={self.status}, bot_name={self.bot_name}, meeting_url={self.meeting_url})"
 
     def _update_attributes(self, data: dict) -> None:
         """Update instance attributes from API response data.