Make conversation name -> id translation more efficient

Author: eizengan

lib/slack/web/api/mixins/conversations.id.rb

@@ -16,7 +16,7 @@ def conversations_id(options = {})
             name = options[:channel]
             raise ArgumentError, 'Required arguments :channel missing' if name.nil?
 
-            id_for :channel, name, '#', :conversations_list, :channels, 'channel_not_found'
+            id_for :channel, name, '#', :conversations_list, :channels, 'channel_not_found', enum_method_options: { limit: 1_000 }
           end
         end
       end

lib/slack/web/api/mixins/ids.id.rb

@@ -6,10 +6,10 @@ module Mixins
         module Ids
           private
 
-          def id_for(key, name, prefix, enum_method, list_method, not_found_error)
+          def id_for(key, name, prefix, enum_method, list_method, not_found_error, enum_method_options: {})
             return { 'ok' => true, key.to_s => { 'id' => name } } unless name[0] == prefix
 
-            public_send enum_method do |list|
+            public_send(enum_method, **enum_method_options) do |list|
               list.public_send(list_method).each do |li|
                 return Slack::Messages::Message.new('ok' => true, key.to_s => { 'id' => li.id }) if li.name == name[1..-1]
               end