Support an array of channels.

Author: dblock

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2025-02-08 14:45:33 UTC using RuboCop version 1.26.1.
+# on 2025-02-08 14:50:37 UTC using RuboCop version 1.26.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -47,7 +47,7 @@ Lint/RedundantCopDisableDirective:
 # Offense count: 13
 # Configuration parameters: IgnoredMethods, CountRepeatedAttributes.
 Metrics/AbcSize:
-  Max: 38
+  Max: 42
 
 # Offense count: 5
 # Configuration parameters: IgnoredMethods.

README.md

@@ -54,11 +54,11 @@ A Ruby client for the Slack [Web](https://api.slack.com/web), [RealTime Messagin
     - [Verifying the Request Signature](#verifying-the-request-signature)
   - [Message Handling](#message-handling)
     - [Formatting Messages](#formatting-messages)
-      - [Date and time formatting](#date-and-time-formatting)
-      - [Channel ID formatting](#channel-id-formatting)
-      - [User ID formatting](#user-id-formatting)
-      - [URL formatting](#url-formatting)
-      - [Markdown formatting](#markdown-formatting)
+      - [Date and Time Formatting](#date-and-time-formatting)
+      - [Channel ID Formatting](#channel-id-formatting)
+      - [User ID Formatting](#user-id-formatting)
+      - [URL Formatting](#url-formatting)
+      - [Markdown Formatting](#markdown-formatting)
     - [Parsing Messages](#parsing-messages)
       - [Unescaping message content](#unescaping-message-content)
       - [Escaping message content](#escaping-message-content)
@@ -189,6 +189,9 @@ client.files_upload_v2(
   thread_ts: '1738331487.481469' # specifies a thread to add this file to
 )
 ```
+
+You can use all of channel ID, an array of channel IDs, or a channel name (prefixed with `#`) in `files_upload_v2`. Lookup by name is not supported by the Slack API and this method called invokes `conversations_list` in order to locate the channel ID. This invocation can have a cost if you have many Slack channels and is only recommended when you intend to list channels anyway.
+
 Note: This library includes a `files_upload` method that uses a deprecated endpoint `files.upload` that will [no longer be supported on 3/11/2025](https://api.slack.com/methods/files.upload#markdown).
 
 ```ruby
@@ -613,15 +616,15 @@ The `verify!` call may raise `Slack::Events::Request::MissingSigningSecret`, `Sl
 
 ### Message Handling
 
-All text in Slack uses the same [system of formatting and escaping](https://api.slack.com/docs/formatting): chat messages, direct messages, file comments, etc. [Slack::Messages::Formatting](lib/slack/messages/formatting.rb) provides convenience methods to format and parse messages.
+All text in Slack uses the same [system of Formatting and escaping](https://api.slack.com/docs/formatting): chat messages, direct messages, file comments, etc. [Slack::Messages::Formatting](lib/slack/messages/formatting.rb) provides convenience methods to format and parse messages.
 
 #### Formatting Messages
 
-`Slack::Messages::Formatting` provides a number of methods for formatting objects that you can then embed in outgoing messages.
+`Slack::Messages::Formatting` provides a number of methods for Formatting objects that you can then embed in outgoing messages.
 
-##### Date and time formatting
+##### Date and Time Formatting
 
-You can embed a pre-formatted date in a message as a string like any other text, but using Slack's date formatting allows you to display dates based on user preferences for dates and times, incorporating users' local time zones, and optionally using relative values like "yesterday", "today", or "tomorrow" when appropriate.
+You can embed a pre-formatted date in a message as a string like any other text, but using Slack's date Formatting allows you to display dates based on user preferences for dates and times, incorporating users' local time zones, and optionally using relative values like "yesterday", "today", or "tomorrow" when appropriate.
 
 ```ruby
 date = Time.now
@@ -644,7 +647,7 @@ Slack::Messages::Formatting.date(date, text: 'party time!')
   # => "<!date^1688150386^{date_num} {time_secs}|party time!>"
 ```
 
-##### Channel ID formatting
+##### Channel ID Formatting
 
 If you already know the channel name you can just embed it in the message as `#some-channel`, but if you only have the ID you can embed it using special syntax which Slack will display as the channel name (while respecting channel visibility).
 
@@ -654,7 +657,7 @@ Slack::Messages::Formatting.channel_link(channel_id)
   # => "<#C0000000001>"
 ```
 
-##### User ID formatting
+##### User ID Formatting
 
 If you already know the user name you can just embed it in the message as `@some_username`, but if you only have the ID you can embed it using special syntax which Slack will display as the user name.
 
@@ -664,9 +667,9 @@ Slack::Messages::Formatting.user_link(user_id)
   # => "<@U0000000001>"
 ```
 
-##### URL formatting
+##### URL Formatting
 
-Slack will automatically parse fully qualified URLs in messages, but you need special formatting to embed a link with different text.
+Slack will automatically parse fully qualified URLs in messages, but you need special Formatting to embed a link with different text.
 
 ```ruby
 text = 'party time'
@@ -675,9 +678,9 @@ Slack::Messages::Formatting.url_link(text, url)
   # => "<https://media.giphy.com/media/AcfTF7tyikWyroP0x7/giphy.gif|party time>"
 ```
 
-##### Markdown formatting
+##### Markdown Formatting
 
-Slack uses a mishmash of regular markdown formatting with its own syntax. Some features like headings aren't supported and will be left as-is, but others like bold, strikethrough, and links are converted.
+Slack uses a mishmash of regular markdown Formatting with its own syntax. Some features like headings aren't supported and will be left as-is, but others like bold, strikethrough, and links are converted.
 
 ```ruby
 text = """

lib/slack/web/api/helpers/files.rb

@@ -32,7 +32,10 @@ def files_upload_v2(params = {})
               raise ArgumentError, "Required argument :#{param} missing" if params[param].nil?
             end
 
-            channel = conversations_id(channel: params[:channels])['channel']['id']
+            channels = Array(params[:channels]).map do |channel|
+              conversations_id(channel: channel)['channel']['id']
+            end.uniq.join(',')
+
             content = params[:content]
             title = params[:title] || params[:filename]
 
@@ -57,7 +60,7 @@ def files_upload_v2(params = {})
 
             # Complete the upload.
             complete_upload_request_params = params.slice(:initial_comment, :thread_ts)
-            complete_upload_request_params[:channels] = channel
+            complete_upload_request_params[:channels] = channels
             complete_upload_request_params[:files] = [{ id: file_id, title: title }].to_json
 
             files_completeUploadExternal(complete_upload_request_params)

spec/slack/web/api/endpoints/custom_specs/files_spec.rb

@@ -80,4 +80,25 @@
       ).files.size).to eq 1
     end
   end
+
+  context 'with an array of channels', vcr: { cassette_name: 'web/files_upload_v2_with_channels_list' } do
+    before do
+      allow(client).to receive(:conversations_list).and_yield(
+        Slack::Messages::Message.new(
+          'channels' => [{
+            'id' => 'C08BHPZBZ8A',
+            'name' => 'general'
+          }]
+        )
+      )
+    end
+
+    it 'completes the upload' do
+      expect(client.files_upload_v2(
+        filename: 'test.txt',
+        content: 'Test File Contents',
+        channels: ['C08AZ76CA4V', '#general']
+      ).files.size).to eq 1
+    end
+  end
 end