slack-smart-bot 1.13.1 → 1.14.0

data/lib/slack/smart-bot/commands/general/public_holidays.rb

@@ -1,55 +1,43 @@
-def public_holidays(country_name, location, year, month, day, add_stats: true, publish_results: true)
-  save_stats(__method__) if add_stats
-  if config[:public_holidays][:api_key].to_s == ""
-    respond "Sorry, I don't have the API key for the public holidays #{config[:public_holidays][:host]}. Set it up on your SmartBot config file."
-  else
-    begin
-      found_location = true
-      http = NiceHttp.new("#{config[:public_holidays][:host]}/api/v2")
-      if !defined?(@countries_candelarific)
-        if File.exist?("#{config.path}/vacations/countries_candelarific.json")
-          @countries_candelarific = JSON.parse(File.read("#{config.path}/vacations/countries_candelarific.json"))
-        else
-          response = http.get "/countries?api_key=#{config[:public_holidays][:api_key]}"
-          countries_candelarific = response.data.json(:countries)
-          if countries_candelarific.is_a?(Array)
-            File.write("#{config.path}/vacations/countries_candelarific.json", countries_candelarific.to_json)
-            @countries_candelarific = JSON.parse(countries_candelarific.to_json)
+class SlackSmartBot
+
+  def public_holidays(country_name, location, year, month, day, add_stats: true, publish_results: true)
+    save_stats(__method__) if add_stats
+    if config[:public_holidays][:api_key].to_s == ""
+      respond "Sorry, I don't have the API key for the public holidays #{config[:public_holidays][:host]}. Set it up on your SmartBot config file."
+    else
+      begin
+        found_location = true
+        http = NiceHttp.new("#{config[:public_holidays][:host]}/api/v2")
+        if !defined?(@countries_candelarific)
+          if File.exist?("#{config.path}/vacations/countries_candelarific.json")
+            @countries_candelarific = JSON.parse(File.read("#{config.path}/vacations/countries_candelarific.json"))
           else
-            @countries_candelarific = []
-          end
-        end
-      end
-      country = @countries_candelarific.find { |c| c.country_name.match?(/^\s*#{country_name}\s*$/i) }
-      if country.nil?
-        respond "Country #{country_name} not found"
-      else
-        country_original_name = country_name.downcase
-        country_region_id = country_name.downcase
-        country_region_id += "/#{location.downcase}" unless location.empty?
-        country_name = country["country_name"]
-        country_iso = country["iso-3166"]
-        states = []
-        if @public_holidays.key?(country_region_id) and @public_holidays[country_region_id].key?(year.to_s)
-          holidays = @public_holidays[country_region_id][year.to_s]
-        elsif File.exist?(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"))
-          holidays = (File.read(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"))).json()
-        elsif !location.empty? and File.exist?(File.join(config.path, "vacations", "#{year}_#{country_original_name.gsub("/", "_").gsub(" ", "_")}.json"))
-          holidays = (File.read(File.join(config.path, "vacations", "#{year}_#{country_original_name.gsub("/", "_").gsub(" ", "_")}.json"))).json()
-          holidays.each do |holiday|
-            if holiday.states.is_a?(Array)
-              states << holiday.states.name
+            response = http.get "/countries?api_key=#{config[:public_holidays][:api_key]}"
+            countries_candelarific = response.data.json(:countries)
+            if countries_candelarific.is_a?(Array)
+              File.write("#{config.path}/vacations/countries_candelarific.json", countries_candelarific.to_json)
+              @countries_candelarific = JSON.parse(countries_candelarific.to_json)
             else
-              states << holiday.states
+              @countries_candelarific = []
             end
           end
-          holidays = holidays.select { |h| h.states.is_a?(String) and h.states == "All" or (h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0) }
-          holidays_specific = holidays.select { |h| h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0 }
-          found_location = false if holidays_specific.length == 0
+        end
+        country = @countries_candelarific.find { |c| c.country_name.match?(/^\s*#{country_name}\s*$/i) }
+        if country.nil?
+          respond "Country #{country_name} not found"
         else
-          response = http.get "/holidays?country=#{country_iso}&year=#{year}&day=#{day}&month=#{month}&api_key=#{config[:public_holidays][:api_key]}"
-          holidays = response.data.json(:holidays)
-          if location != ""
+          country_original_name = country_name.downcase
+          country_region_id = country_name.downcase
+          country_region_id += "/#{location.downcase}" unless location.empty?
+          country_name = country["country_name"]
+          country_iso = country["iso-3166"]
+          states = []
+          if @public_holidays.key?(country_region_id) and @public_holidays[country_region_id].key?(year.to_s)
+            holidays = @public_holidays[country_region_id][year.to_s]
+          elsif File.exist?(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"))
+            holidays = (File.read(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"))).json()
+          elsif !location.empty? and File.exist?(File.join(config.path, "vacations", "#{year}_#{country_original_name.gsub("/", "_").gsub(" ", "_")}.json"))
+            holidays = (File.read(File.join(config.path, "vacations", "#{year}_#{country_original_name.gsub("/", "_").gsub(" ", "_")}.json"))).json()
             holidays.each do |holiday|
               if holiday.states.is_a?(Array)
                 states << holiday.states.name
@@ -60,85 +48,99 @@ def public_holidays(country_name, location, year, month, day, add_stats: true, p
             holidays = holidays.select { |h| h.states.is_a?(String) and h.states == "All" or (h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0) }
             holidays_specific = holidays.select { |h| h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0 }
             found_location = false if holidays_specific.length == 0
+          else
+            response = http.get "/holidays?country=#{country_iso}&year=#{year}&day=#{day}&month=#{month}&api_key=#{config[:public_holidays][:api_key]}"
+            holidays = response.data.json(:holidays)
+            if location != ""
+              holidays.each do |holiday|
+                if holiday.states.is_a?(Array)
+                  states << holiday.states.name
+                else
+                  states << holiday.states
+                end
+              end
+              holidays = holidays.select { |h| h.states.is_a?(String) and h.states == "All" or (h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0) }
+              holidays_specific = holidays.select { |h| h.states.is_a?(Array) and h.states.name.grep(/^\s*#{location}\s*$/i).length > 0 }
+              found_location = false if holidays_specific.length == 0
+            end
+            if day == "" and month == "" and holidays.is_a?(Array) and holidays.length > 0 and found_location
+              File.write(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"), holidays.to_json) if holidays.is_a?(Array)
+            end
           end
-          if day == "" and month == "" and holidays.is_a?(Array) and holidays.length > 0 and found_location
-            File.write(File.join(config.path, "vacations", "#{year}_#{country_region_id.gsub("/", "_").gsub(" ", "_")}.json"), holidays.to_json) if holidays.is_a?(Array)
+          if day == "" and month == ""
+            date = year
+          elsif day == ""
+            date = "#{year}-#{"%02d" % month.to_i}"
+          else
+            date = "#{year}-#{"%02d" % month.to_i}-#{"%02d" % day.to_i}"
           end
-        end
-        if day == "" and month == ""
-          date = year
-        elsif day == ""
-          date = "#{year}-#{"%02d" % month}"
-        else
-          date = "#{year}-#{"%02d" % month}-#{"%02d" % day}"
-        end
-
-        if holidays.is_a?(Array) and holidays.length > 0 and found_location
-          date_holiday = ""
-          messages = ["*Holidays in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}*"]
-          num_holidays_to_show = 0
-          all_holidays = []
-          states = []
-          holidays_to_add = []
-          holidays.each do |holiday|
-            if holiday.type.join.match?(/holiday/i)
-              if location == "" or (location != "" and (holiday.states.is_a?(String) and holiday.states == "All") or (holiday.states.is_a?(Array) and holiday.states.name.grep(/#{location}/i).length > 0))
-                holiday_id = "#{holiday[:date][:datetime][:year]}-#{"%02d" % holiday[:date][:datetime][:month]}-#{"%02d" % holiday[:date][:datetime][:day]} #{holiday[:name]}"
-                unless all_holidays.include?(holiday_id) or
-                       (day != "" and holiday[:date][:datetime][:day] != day.to_i) or
-                       (month != "" and holiday[:date][:datetime][:month] != month.to_i)
-                  all_holidays << holiday_id
-                  if day == ""
-                    m = holiday[:date][:datetime][:month]
-                    d = holiday[:date][:datetime][:day]
-                    date_holiday = " #{holiday[:date][:datetime][:year]}-#{"%02d" % m}-#{"%02d" % d} "
-                  end
-                  num_holidays_to_show += 1
-                  break if num_holidays_to_show > 30 and publish_results
-                  week_day = Date.new(holiday[:date][:datetime][:year], holiday[:date][:datetime][:month], holiday[:date][:datetime][:day]).strftime("%A")
-                  messages << "\t:spiral_calendar_pad:#{date_holiday}*#{holiday[:name]}* _(#{holiday[:type].join(", ")}) (#{week_day})_"
-                  messages << "\t#{holiday[:description]}"
-                  if location == ""
+          if holidays.is_a?(Array) and holidays.length > 0 and found_location
+            date_holiday = ""
+            messages = ["*Holidays in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}*"]
+            num_holidays_to_show = 0
+            all_holidays = []
+            states = []
+            holidays_to_add = []
+            holidays.each do |holiday|
+              if holiday.type.join.match?(/holiday/i)
+                if location == "" or (location != "" and (holiday.states.is_a?(String) and holiday.states == "All") or (holiday.states.is_a?(Array) and holiday.states.name.grep(/#{location}/i).length > 0))
+                  holiday_id = "#{holiday[:date][:datetime][:year]}-#{"%02d" % holiday[:date][:datetime][:month]}-#{"%02d" % holiday[:date][:datetime][:day]} #{holiday[:name]}"
+                  unless all_holidays.include?(holiday_id) or
+                        (day != "" and holiday[:date][:datetime][:day] != day.to_i) or
+                        (month != "" and holiday[:date][:datetime][:month] != month.to_i)
+                    all_holidays << holiday_id
+                    if day == ""
+                      m = holiday[:date][:datetime][:month]
+                      d = holiday[:date][:datetime][:day]
+                      date_holiday = " #{holiday[:date][:datetime][:year]}-#{"%02d" % m}-#{"%02d" % d} "
+                    end
+                    num_holidays_to_show += 1
+                    break if num_holidays_to_show > 30 and publish_results
+                    week_day = Date.new(holiday[:date][:datetime][:year], holiday[:date][:datetime][:month], holiday[:date][:datetime][:day]).strftime("%A")
+                    messages << "\t:spiral_calendar_pad:#{date_holiday}*#{holiday[:name]}* _(#{holiday[:type].join(", ")}) (#{week_day})_"
+                    messages << "\t#{holiday[:description]}"
+                    if location == ""
+                      if holiday.states.is_a?(Array)
+                        messages << "\tLocations: #{holiday.states.name.sort.join(", ")}"
+                      else
+                        messages << "\tLocations: #{holiday.states}"
+                      end
+                    end
                     if holiday.states.is_a?(Array)
-                      messages << "\tLocations: #{holiday.states.name.sort.join(", ")}"
+                      states << holiday.states.name
                     else
-                      messages << "\tLocations: #{holiday.states}"
+                      states << holiday.states
                     end
+                    messages << "\n"
+                    holidays_to_add << holiday
                   end
-                  if holiday.states.is_a?(Array)
-                    states << holiday.states.name
-                  else
-                    states << holiday.states
-                  end
-                  messages << "\n"
-                  holidays_to_add << holiday
                 end
               end
             end
-          end
-          @public_holidays[country_region_id] = {} if !@public_holidays.key?(country_region_id)
-          @public_holidays[country_region_id][year.to_s] = holidays_to_add if !@public_holidays[country_region_id].key?(year.to_s)
+            @public_holidays[country_region_id] = {} if !@public_holidays.key?(country_region_id)
+            @public_holidays[country_region_id][year.to_s] = holidays_to_add if !@public_holidays[country_region_id].key?(year.to_s) and date == year
 
-          if num_holidays_to_show > 30
-            messages = ["*Holidays in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}*"]
-            messages << "Too many holidays to show, please refine your search"
+            if num_holidays_to_show > 30
+              messages = ["*Holidays in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}*"]
+              messages << "Too many holidays to show, please refine your search"
+            end
+          else
+            messages = ["*No holidays found in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}. Be sure the Country and State are written correctly. Try using just the Country, not all countries are supporting States.*"]
           end
-        else
-          messages = ["*No holidays found in #{country_name}#{" #{location.downcase.capitalize}" unless location.empty?} in #{date}. Be sure the Country and State are written correctly. Try using just the Country, not all countries are supporting States.*"]
-        end
-        states.flatten!
-        states.uniq!
-        states.delete("All")
-        if states.length > 1 and (location == "" or !found_location)
-          messages << "*All States found in #{date} #{country_name}*: #{states.sort.join(", ")}"
-          respond messages[-1] if !publish_results
+          states.flatten!
+          states.uniq!
+          states.delete("All")
+          if states.length > 1 and (location == "" or !found_location)
+            messages << "*All States found in #{date} #{country_name}*: #{states.sort.join(", ")}"
+            respond messages[-1] if !publish_results
+          end
+          respond messages.join("\n") unless !publish_results
         end
-        respond messages.join("\n") unless !publish_results
+      rescue Exception => stack
+        respond "Sorry, I can't get the public holidays for #{country_name} #{location} in #{date}. Error: #{stack.message}"
+        @logger.fatal stack
       end
-    rescue Exception => stack
-      respond "Sorry, I can't get the public holidays for #{country_name} #{location} in #{date}. Error: #{stack.message}"
-      @logger.fatal stack
+      return (found_location==true and !country.nil?) if !publish_results
     end
-    return (found_location==true and !country.nil?) if !publish_results
   end
-end
+end

data/lib/slack/smart-bot/commands/general/set_public_holidays.rb

@@ -9,13 +9,17 @@ class SlackSmartBot
         else
             country_region = "#{country}/#{state}"
         end
-        respond "Public holidays for *#{country_region}* set."
+        if state == ''
+          respond "Public holidays for *#{country_region}* set. If available States, try with the country and state to be more precise."
+        else
+          respond "Public holidays for *#{country_region}* set."
+        end
         get_vacations()
         @vacations[user.name] ||= {}
         @vacations[user.name][:public_holidays] = country_region
         update_vacations()
     else
-        respond "Be sure the country and state are correct."
+        respond "Be sure the country and state are correct. If not displayed available states, try with the country only."
     end
   end
 end

data/lib/slack/smart-bot/commands/general/teams/add_team.rb

@@ -0,0 +1,87 @@
+class SlackSmartBot
+  module Commands
+    module General
+      module Teams
+        def add_team(user, name, options, info)
+          save_stats(__method__)
+
+          get_teams()
+          if @teams.key?(name.to_sym)
+            respond "It seems like the team *#{name}* already exists.\nRelated commands `update team TEAM_NAME PROPERTIES`, `delete team TEAM_NAME`, `see team TEAM_NAME`, `see teams`"
+          else
+            wrong = false
+            team = { members: {}, channels: {} }
+            last_type = nil
+            type_detected = false
+            options.split(/\s+/).each do |opt|
+              type_detected = false
+              if opt.match?(/^\s*$/)
+                #blank
+              elsif opt.match?(/^[\w\-]+$/i)
+                last_type = opt
+                type_detected = true
+              elsif opt.match(/<@(\w+)>/i)
+                team[:members][last_type] ||= []
+                if last_type.nil?
+                  wrong = true
+                  respond "You need to specify the TYPE for the member."
+                  break
+                else
+                  member_id = $1
+                  member_info = @users.select { |u| u.id == member_id or (u.key?(:enterprise_user) and u.enterprise_user.id == member_id) }[-1]
+                  if member_info.nil?
+                    @users = get_users()
+                    member_info = @users.select { |u| u.id == member_id or (u.key?(:enterprise_user) and u.enterprise_user.id == member_id) }[-1]
+                  end
+                  team[:members][last_type] << member_info.name
+                end
+              elsif opt.match(/<#(\w+)\|[^>]*>/i)
+                team[:channels][last_type] ||= []
+                if last_type.nil?
+                  wrong = true
+                  respond "You need to specify the TYPE for the channel."
+                  break
+                else
+                  channel_id = $1
+                  get_channels_name_and_id() unless @channels_name.keys.include?(channel_id)
+                  channel = @channels_name[channel_id]
+                  channel_members = get_channel_members(channel_id) unless channel.nil?
+                  if channel.nil? or !channel_members.include?(config.nick_id)
+                    respond ":exclamation: Add the Smart Bot to *<##{channel_id}>* channel first."
+                    wrong = true
+                    break
+                  else
+                    team[:channels][last_type] << channel
+                  end
+                end
+              else
+                respond "It seems like the members or channel list is not correct. Please double check."
+                wrong = true
+                break
+              end
+            end
+            if type_detected #type added but not added a channel or user
+              respond "It seems like the parameters supplied are not correct. Please double check."
+              wrong = true
+            end
+
+            unless wrong
+              get_teams()
+              team[:info] = info
+              team[:status] = :added
+              team[:user] = user.name
+              team[:creator] = user.name
+              team[:date] = Time.now.strftime("%Y-%m-%dT%H:%M:%S.000Z")[0..18]
+              new_team = {}
+              team[:name] = name
+              new_team[name.to_sym] = team
+              update_teams(new_team)
+              respond "The *#{name}* team has been added."
+              see_teams(user, name, add_stats: false)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/slack/smart-bot/commands/general/teams/delete_team.rb

@@ -0,0 +1,69 @@
+class SlackSmartBot
+  module Commands
+    module General
+      module Teams
+        def delete_team(user, team_name)
+          save_stats(__method__) if answer.empty?
+
+          if Thread.current[:dest][0] == "D"
+            respond "This command cannot be called from a DM"
+          else
+            get_teams()
+            if @teams.key?(team_name.to_sym)
+              assigned_members = @teams[team_name.to_sym].members.values.flatten
+              assigned_members.uniq!
+              all_team_members = assigned_members.dup
+              team_members = []
+              if @teams[team_name.to_sym].channels.key?("members")
+                @teams[team_name.to_sym].channels["members"].each do |ch|
+                  get_channels_name_and_id() unless @channels_id.key?(ch)
+                  tm = get_channel_members(@channels_id[ch])
+                  tm.each do |m|
+                    user_info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) }[-1]
+                    team_members << user_info.name unless user_info.is_app_user or user_info.is_bot
+                  end
+                end
+              end
+              team_members.flatten!
+              team_members.uniq!
+              all_team_members += team_members
+              all_team_members.uniq!
+            end
+            if !@teams.key?(team_name.to_sym)
+              respond "It seems like the team *#{team_name}* doesn't exist.\nRelated commands `add team TEAM_NAME PROPERTIES`, `see team TEAM_NAME`, `see teams`"
+            elsif !(all_team_members + [@teams[team_name.to_sym].creator] + config.masters).flatten.include?(user.name)
+              respond "You have to be a member of the team, the creator or a Master admin to be able to delete this team."
+            else
+              if answer.empty?
+                ask "do you really want to delete the #{team_name} team? (yes/no)"
+              else
+                case answer
+                when /yes/i, /yep/i, /sure/i
+                  answer_delete
+                  @teams.delete(team_name.to_sym)
+                  require "fileutils"
+                  time_deleted = Time.now.strftime("%Y%m%d%H%M%S")
+                  FileUtils.mv(File.join(config.path, "teams", "t_#{team_name}.yaml"),
+                               File.join(config.path, "teams", "t_#{team_name}_#{time_deleted}.yaml.deleted"))
+                  deleted_memos_file = File.join(config.path, "teams", "t_#{team_name}_memos.yaml.deleted")
+                  if File.exist?(deleted_memos_file)
+                    FileUtils.mv(deleted_memos_file,
+                                 File.join(config.path, "teams", "t_#{team_name}_memos_#{time_deleted}.yaml.deleted"))
+                  end
+                  update_teams()
+                  respond "The team #{team_name} has been deleted."
+                when /no/i, /nope/i, /cancel/i
+                  answer_delete
+                  respond "Ok, the team was not deleted"
+                else
+                  respond "I don't understand"
+                  ask "do you really want to delete the #{team_name} team? (yes/no)"
+                end
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/slack/smart-bot/commands/general/teams/memos/add_memo_team.rb

@@ -0,0 +1,136 @@
+class SlackSmartBot
+  module Commands
+    module General
+      module Teams
+        module Memos
+          def add_memo_team(user, privacy, team_name, topic, type, message)
+            save_stats(__method__)
+
+            get_teams()
+            if @teams.key?(team_name.to_sym)
+              assigned_members = @teams[team_name.to_sym].members.values.flatten
+              assigned_members.uniq!
+              all_team_members = assigned_members.dup
+              team_members = []
+              if @teams[team_name.to_sym].channels.key?("members")
+                @teams[team_name.to_sym].channels["members"].each do |ch|
+                  get_channels_name_and_id() unless @channels_id.key?(ch)
+                  tm = get_channel_members(@channels_id[ch])
+                  tm.each do |m|
+                    user_info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) }[-1]
+                    team_members << user_info.name unless user_info.is_app_user or user_info.is_bot
+                  end
+                end
+              end
+              team_members.flatten!
+              team_members.uniq!
+              all_team_members += team_members
+              all_team_members.uniq!
+            end
+            search = false
+            if type == "jira"
+              able_to_connect_jira = false
+              begin
+                http = NiceHttp.new(config.jira.host)
+                http.headers.authorization = NiceHttpUtils.basic_authentication(user: config.jira.user, password: config.jira.password)
+                message.gsub!(/^\s*</, "")
+                message.gsub!(/\>$/, "")
+                message.gsub!(/\|.+$/, "")
+                message.gsub!(/^#{config.jira.host}/, "")
+                if message.include?("/browse/")
+                  message = message.scan(/\/browse\/(.+)/).join
+                  resp = http.get("/rest/api/latest/issue/#{message}")
+                else
+                  message.gsub!(/^\/issues\/\?jql=/, "")
+                  message.gsub!(" ", "%20")
+                  resp = http.get("/rest/api/latest/search/?jql=#{message}")
+                  search = true
+                end
+                if resp.code == 200
+                  able_to_connect_jira = true
+                else
+                  error_code = resp.code
+                  if resp.code == 400
+                    error_message = resp.data.json(:errorMessages)[-1]
+                  else
+                    error_message = ""
+                  end
+                end
+                http.close
+              rescue => exception
+                @logger.fatal exception
+              end
+            elsif type == "github"
+              able_to_connect_github = false
+              begin
+                search = true if message.include?("?")
+                http = NiceHttp.new(config.github.host)
+                http.headers.authorization = "token #{config.github.token}"
+                message.gsub!(/^\s*</, "")
+                message.gsub!(/\>$/, "")
+                message.gsub!(/\|.+$/, "")
+                message.gsub!(/^#{config.github.host}/, "")
+                message.gsub!("https://github.com", "")
+                message.slice!(0) if message[0] == "/"
+                resp = http.get("/repos/#{message}")
+                if resp.code == 200
+                  able_to_connect_github = true
+                else
+                  error_code = resp.code
+                  if resp.code == 401
+                    error_message = resp.data.json(:message)[-1]
+                  else
+                    error_message = ""
+                  end
+                end
+                http.close
+              rescue => exception
+                @logger.fatal exception
+              end
+            end
+
+            if !@teams.key?(team_name.to_sym)
+              respond "It seems like the team *#{team_name}* doesn't exist\nRelated commands `add team TEAM_NAME PROPERTIES`, `see team TEAM_NAME`, `see teams`"
+            elsif !(all_team_members + config.masters).flatten.include?(user.name)
+              respond "You have to be a member of the team or a Master admin to be able to add a memo to the team."
+            elsif type == "jira" and !able_to_connect_jira
+              if error_message == ""
+                respond "You need to supply the correct credentials for JIRA on the SmartBot settings: `jira: { host: HOST, user: USER, password: PASSWORD }` and a correct JQL string or JQL url"
+              else
+                respond "You need to supply a correct JQL string or JQL url: #{error_message}"
+              end
+            else
+              topic = :no_topic if topic == ""
+              @teams[team_name.to_sym][:memos] ||= []
+              if @teams[team_name.to_sym][:memos].empty? and !@teams[team_name.to_sym].key?(:last_memo_id)
+                memo_id = 1
+              else
+                if @teams[team_name.to_sym].key?(:last_memo_id)
+                  memo_id = @teams[team_name.to_sym][:last_memo_id] + 1
+                else
+                  memo_id = @teams[team_name.to_sym][:memos].memo_id.flatten.max + 1 #backwards compatibility
+                end
+              end
+              @teams[team_name.to_sym][:last_memo_id] = memo_id
+              @teams[team_name.to_sym][:memos] << {
+                memo_id: memo_id,
+                topic: topic,
+                type: type,
+                privacy: privacy,
+                user: user.name,
+                date: Time.now.strftime("%Y-%m-%dT%H:%M:%S.000Z")[0..18],
+                message: message,
+                status: ":new: ",
+                search: search,
+                issues: [],
+                comments: [],
+              }
+              update_teams()
+              respond "The memo has been added to *#{team_name}* team. (#{memo_id})"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/slack/smart-bot/commands/general/teams/memos/add_memo_team_comment.rb

@@ -0,0 +1,37 @@
+class SlackSmartBot
+  module Commands
+    module General
+      module Teams
+        module Memos
+          def add_memo_team_comment(user, team_name, memo_id, message)
+            save_stats(__method__)
+
+            get_teams()
+            team_name = team_name.to_sym
+            if @teams.key?(team_name)
+              if @teams[team_name].key?(:memos)
+                memo = @teams[team_name].memos.select { |m| m.memo_id == memo_id.to_i }[-1]
+                if memo
+                  memo.comments ||= []
+                  memo.comments << { user_name: user.name, user_id: user.id, message: message, time: Time.now.to_s }
+                  update_teams()
+                  if config.simulate
+                    respond "Comment added to memo #{memo_id} in team #{team_name}"
+                  else
+                    react :spiral_note_pad
+                  end
+                else
+                  respond "Memo *#{memo_id}* does not exist in team *#{team_name}*."
+                end
+              else
+                respond "There are no memos in team *#{team_name}*."
+              end
+            else
+              respond "Team *#{team_name}* does not exist. Call `see teams` to see the list of teams."
+            end
+          end
+        end
+      end
+    end
+  end
+end