From d0db05a821edd6cd35492ecf46fd02059c3026ba Mon Sep 17 00:00:00 2001 From: Kazuhiro Sera Date: Fri, 25 Feb 2022 07:13:52 +0900 Subject: [PATCH] Add file_ids to chat.update parameters (#1187) --- slack_sdk/web/async_client.py | 7 ++++++- slack_sdk/web/client.py | 7 ++++++- slack_sdk/web/legacy_client.py | 7 ++++++- 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/slack_sdk/web/async_client.py b/slack_sdk/web/async_client.py index ea470a42c..5995e53c0 100644 --- a/slack_sdk/web/async_client.py +++ b/slack_sdk/web/async_client.py @@ -2211,6 +2211,7 @@ async def chat_update( attachments: Optional[Sequence[Union[Dict, Attachment]]] = None, blocks: Optional[Sequence[Union[Dict, Block]]] = None, as_user: Optional[bool] = None, + file_ids: Optional[Union[str, Sequence[str]]] = None, link_names: Optional[bool] = None, parse: Optional[str] = None, # none, full reply_broadcast: Optional[bool] = None, @@ -2232,6 +2233,10 @@ async def chat_update( "reply_broadcast": reply_broadcast, } ) + if isinstance(file_ids, (list, Tuple)): + kwargs.update({"file_ids": ",".join(file_ids)}) + else: + kwargs.update({"file_ids": file_ids}) _parse_web_class_objects(kwargs) kwargs = _remove_none_values(kwargs) _warn_if_text_is_missing("chat.update", kwargs) @@ -2448,7 +2453,7 @@ async def conversations_inviteShared( else: kwargs.update({"emails": emails}) if isinstance(user_ids, (list, Tuple)): - kwargs.update({"emails": ",".join(user_ids)}) + kwargs.update({"user_ids": ",".join(user_ids)}) else: kwargs.update({"user_ids": user_ids}) return await self.api_call( diff --git a/slack_sdk/web/client.py b/slack_sdk/web/client.py index 0eb2307de..373a61a85 100644 --- a/slack_sdk/web/client.py +++ b/slack_sdk/web/client.py @@ -2160,6 +2160,7 @@ def chat_update( attachments: Optional[Sequence[Union[Dict, Attachment]]] = None, blocks: Optional[Sequence[Union[Dict, Block]]] = None, as_user: Optional[bool] = None, + file_ids: Optional[Union[str, Sequence[str]]] = None, link_names: Optional[bool] = None, parse: Optional[str] = None, # none, full reply_broadcast: Optional[bool] = None, @@ -2181,6 +2182,10 @@ def chat_update( "reply_broadcast": reply_broadcast, } ) + if isinstance(file_ids, (list, Tuple)): + kwargs.update({"file_ids": ",".join(file_ids)}) + else: + kwargs.update({"file_ids": file_ids}) _parse_web_class_objects(kwargs) kwargs = _remove_none_values(kwargs) _warn_if_text_is_missing("chat.update", kwargs) @@ -2395,7 +2400,7 @@ def conversations_inviteShared( else: kwargs.update({"emails": emails}) if isinstance(user_ids, (list, Tuple)): - kwargs.update({"emails": ",".join(user_ids)}) + kwargs.update({"user_ids": ",".join(user_ids)}) else: kwargs.update({"user_ids": user_ids}) return self.api_call( diff --git a/slack_sdk/web/legacy_client.py b/slack_sdk/web/legacy_client.py index d4109b2f9..d5b5c910f 100644 --- a/slack_sdk/web/legacy_client.py +++ b/slack_sdk/web/legacy_client.py @@ -2171,6 +2171,7 @@ def chat_update( attachments: Optional[Sequence[Union[Dict, Attachment]]] = None, blocks: Optional[Sequence[Union[Dict, Block]]] = None, as_user: Optional[bool] = None, + file_ids: Optional[Union[str, Sequence[str]]] = None, link_names: Optional[bool] = None, parse: Optional[str] = None, # none, full reply_broadcast: Optional[bool] = None, @@ -2192,6 +2193,10 @@ def chat_update( "reply_broadcast": reply_broadcast, } ) + if isinstance(file_ids, (list, Tuple)): + kwargs.update({"file_ids": ",".join(file_ids)}) + else: + kwargs.update({"file_ids": file_ids}) _parse_web_class_objects(kwargs) kwargs = _remove_none_values(kwargs) _warn_if_text_is_missing("chat.update", kwargs) @@ -2406,7 +2411,7 @@ def conversations_inviteShared( else: kwargs.update({"emails": emails}) if isinstance(user_ids, (list, Tuple)): - kwargs.update({"emails": ",".join(user_ids)}) + kwargs.update({"user_ids": ",".join(user_ids)}) else: kwargs.update({"user_ids": user_ids}) return self.api_call(