slack-smart-bot 1.11.0 → 1.12.1

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

@@ -5,12 +5,12 @@ class SlackSmartBot
     get_teams()
     teams = @teams.deep_copy
     if teams.empty?
-      respond "There are no teams added yet. Use `add team` command to add a team."      
-    elsif team_name.to_s != "" and !teams.key?(team_name.to_sym) and (teams.keys.select {|t| (t.to_s.gsub('-','').gsub('_','') == team_name.to_s)}).empty?
+      respond "There are no teams added yet. Use `add team` command to add a team."
+    elsif team_name.to_s != "" and !teams.key?(team_name.to_sym) and (teams.keys.select { |t| (t.to_s.gsub("-", "").gsub("_", "") == team_name.to_s) }).empty?
       respond "It seems like the team *#{team_name}* doesn't exist.\nRelated commands `add team TEAM_NAME PROPERTIES`, `see team TEAM_NAME`, `see teams`"
     else
       users_link = (Thread.current[:dest][0] == "D")
-      filter = (search != '')
+      filter = (search != "")
       react :runner
       @users = get_users() if add_stats
 
@@ -19,7 +19,7 @@ class SlackSmartBot
       search_channels = []
       search_info = []
       if filter
-        search.split(' ').each do |s|
+        search.split(" ").each do |s|
           if s.match(/<@(\w+)>/i)
             m = $1
             user_info = @users.select { |u| u.id.downcase == m.downcase or (u.key?(:enterprise_user) and u.enterprise_user.id.downcase == m.downcase) }[-1]
@@ -29,28 +29,28 @@ class SlackSmartBot
             search_channels << @channels_name[c] if @channels_name.key?(c)
           else
             search_info << s
-          end  
+          end
         end
       end
-      if team_name.to_s == '' and search.to_s == ''
+      if team_name.to_s == "" and search.to_s == ""
         dest = :on_thread
-        messages.unshift('Since there are many lines returned the results are returned on a thread by default.')
+        messages.unshift("Since there are many lines returned the results are returned on a thread by default.")
       else
         dest = Thread.current[:dest]
       end
 
       teams.each do |name, team|
         filter ? add = false : add = true
-        if team_name.to_s == "" or (team_name.to_s == name.to_s) or (name.to_s.gsub('-','').gsub('_','') == team_name.to_s)
+        if team_name.to_s == "" or (team_name.to_s == name.to_s) or (name.to_s.gsub("-", "").gsub("_", "") == team_name.to_s)
           message = []
           message << "*#{name.capitalize}*"
 
           if filter and search_info.size > 0
             all_info = true
             search_info.each do |s|
-              if (team.members.keys.find { |e| /#{s}/i =~ e})
-                add = true 
-                break              
+              if (team.members.keys.find { |e| /#{s}/i =~ e })
+                add = true
+                break
               end
               if !name.match?(/#{s}/i)
                 all_info = false
@@ -68,7 +68,8 @@ class SlackSmartBot
             user_info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) or u.name == m or (u.key?(:enterprise_user) and u.enterprise_user.name == m) }[-1]
             assigned_members.delete(m) if user_info.nil? or user_info.deleted
           end
-
+          channels_members = []
+          all_team_members = assigned_members.dup
           if team.channels.key?("members")
             team_members = []
             team.channels["members"].each do |ch|
@@ -77,6 +78,7 @@ class SlackSmartBot
               if tm.nil?
                 respond ":exclamation: Add the Smart Bot to *##{ch}* channel to be able to get the list of members.", dest
               else
+                channels_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
@@ -88,6 +90,7 @@ class SlackSmartBot
             unassigned_members = team_members - assigned_members
             unassigned_members.delete(config.nick)
             not_on_team_channel = assigned_members - team_members
+            all_team_members += team_members
           else
             unassigned_members = []
             not_on_team_channel = []
@@ -96,7 +99,7 @@ class SlackSmartBot
             um = unassigned_members.dup
             um.each do |m|
               user_info = @users.select { |u| u.name == m or (u.key?(:enterprise_user) and u.enterprise_user.name == m) }[-1]
-              unless user_info.nil? or user_info.profile.title.to_s==''
+              unless user_info.nil? or user_info.profile.title.to_s == ""
                 team.members[user_info.profile.title.to_snake_case] ||= []
                 team.members[user_info.profile.title.to_snake_case] << m
                 unassigned_members.delete(m)
@@ -117,9 +120,9 @@ class SlackSmartBot
           if filter and search_info.size > 0
             all_info = true
             search_info.each do |s|
-              if (team.members.keys.find { |e| /#{s}/i =~ e})
-                add = true 
-                break              
+              if (team.members.keys.find { |e| /#{s}/i =~ e })
+                add = true
+                break
               end
               if !team.info.match?(/#{s}/i)
                 all_info = false
@@ -134,7 +137,7 @@ class SlackSmartBot
               team.members.each do |type, members|
                 message << "        _`#{type}`_:  "
                 members.each do |member|
-                  types = [":palm_tree:", ":spiral_calendar_pad:", ":face_with_thermometer:"]
+                  types = [":palm_tree:", ":spiral_calendar_pad:", ":face_with_thermometer:", ":baby:"]
                   member_info = @users.select { |u| u.name == member }[-1]
                   if !member_info.nil? and !member_info.deleted
                     member_id = member_info.id
@@ -146,7 +149,7 @@ class SlackSmartBot
                       active = (get_presence(member_id).presence.to_s == "active")
                       if active
                         user_info = @users.select { |u| u.id == member_id or (u.key?(:enterprise_user) and u.enterprise_user.name == member_id) }[-1]
-                        if (user_info.tz_offset-user.tz_offset).abs <= (4*3600)
+                        if (user_info.tz_offset - user.tz_offset).abs <= (4 * 3600)
                           status = ":large_green_circle:"
                         else
                           status = ":large_yellow_circle:"
@@ -158,19 +161,19 @@ class SlackSmartBot
                   else
                     status = ":exclamation:"
                   end
-                  unless status == ':exclamation:'
+                  unless status == ":exclamation:"
                     if users_link
                       message[-1] += "  #{status}<@#{member}>, "
                     else
-                        user_info = @users.select { |u| u.name == member or (u.key?(:enterprise_user) and u.enterprise_user.name == member) }[-1]
-                        unless user_info.nil?
-                          if user_info.profile.display_name == ''
-                            name = user_info.name
-                          else
-                            name = user_info.profile.display_name
-                          end
-                          message[-1] += "  #{status} #{name}, "
+                      user_info = @users.select { |u| u.name == member or (u.key?(:enterprise_user) and u.enterprise_user.name == member) }[-1]
+                      unless user_info.nil?
+                        if user_info.profile.display_name == ""
+                          name = user_info.name
+                        else
+                          name = user_info.profile.display_name
                         end
+                        message[-1] += "  #{status} #{name}, "
+                      end
                     end
                   end
                 end
@@ -186,7 +189,7 @@ class SlackSmartBot
                   members.each do |m|
                     user_info = @users.select { |u| u.name == m or (u.key?(:enterprise_user) and u.enterprise_user.name == m) }[-1]
                     unless user_info.nil? or user_info.deleted
-                      if user_info.profile.display_name == ''
+                      if user_info.profile.display_name == ""
                         name = user_info.name
                       else
                         name = user_info.profile.display_name
@@ -200,20 +203,164 @@ class SlackSmartBot
             end
           end
 
-
           if add
             message << "   > *_channels_*"
             team.channels.each do |type, channels|
               channel_ids = []
               channels.each do |ch|
-                channel_info = @channels_list.select { |c| c.name.to_s.downcase == ch.to_s.downcase}[-1]
-                if @channels_id.key?(ch) and (!channel_info.is_private or (channel_info.is_private and (team.members.values+[team.creator]).flatten.include?(user.name)))
+                channel_info = @channels_list.select { |c| c.name.to_s.downcase == ch.to_s.downcase }[-1]
+                if @channels_id.key?(ch) and (!channel_info.is_private or (channel_info.is_private and (team.members.values + [team.creator]).flatten.include?(user.name)))
                   channel_ids << @channels_id[ch]
                 end
               end
               message << "        _`#{type}`_:  <##{channel_ids.join("> <#")}>" unless channel_ids.empty?
             end
 
+            unless !team.key?(:memos) or team.memos.empty? or (team_name.to_s == "" and search.to_s == "")
+              all_memos = {}
+              team.memos.each do |memo|
+                if memo.privacy.empty? or
+                   (memo.privacy == "private" and (all_team_members.include?(user.name) and (users_link or channels_members.include?(Thread.current[:dest])))) or
+                   (memo.privacy == "personal" and memo.user == user.name and users_link)
+                  if memo.type == "jira" and config.jira.host != ""
+                    http = NiceHttp.new(config.jira.host)
+                    http.headers.authorization = NiceHttpUtils.basic_authentication(user: config.jira.user, password: config.jira.password)
+                    if memo.message.match?(/^\w+\-\d+$/)
+                      resp = http.get("/rest/api/latest/issue/#{memo.message}")
+                      issues = [resp.data.json] if resp.code == 200
+                    else
+                      resp = http.get("/rest/api/latest/search/?jql=#{memo.message}")
+                      issues = resp.data.json().issues if resp.code == 200
+                    end
+                    if resp.code == 200
+                      unless issues.empty?
+                        issues.each do |issue|
+                          jira_memo = { jira: true, github: false, status: "", memo_id: memo.memo_id, topic: memo.topic, privacy: memo.privacy, user: memo.user, date: memo.date, message: "", type: memo.type }
+                          jira_memo.message = issue.fields.summary
+                          jira_memo.user = issue.fields.reporter.name
+                          jira_memo.date = issue.fields.created
+                          if memo.topic == :no_topic and !issue.fields.labels.empty?
+                            jira_memo.topic = issue.fields.labels.sort.join("_").split(" ").join("_")
+                          end
+                          case issue.fields.issuetype.name
+                          when "Story"; jira_memo.type = ":abc:"
+                          when "Bug"; jira_memo.type = ":bug:"
+                          when "Task"; jira_memo.type = ":clock1:"
+                          when "New Feature", "Improvement"; jira_memo.type = ":sunny:"
+                          else jira_memo.type = ":memo:"
+                          end
+                          case issue.fields.status.statusCategory.name
+                          when "Done"; jira_memo.status = ":heavy_check_mark:"
+                          when "To Do"; jira_memo.status = ":new:"
+                          when "In Progress"; jira_memo.status = ":runner:"
+                          else jira_memo.status = ":heavy_minus_sign:"
+                          end
+                          #todo: check if possible to add link to status instead of jira issue
+                          #jira_memo.status = " <#{config.jira.host}/browse/#{issue[:key]}|#{jira_memo.status}> "
+                          jira_memo.status += " <#{config.jira.host}/browse/#{issue[:key]}|#{issue[:key]}>"
+
+                          all_memos[jira_memo.topic] ||= []
+                          all_memos[jira_memo.topic] << jira_memo
+                        end
+                      end
+                    end
+                    http.close
+                  elsif memo.type == "github" and config.github.host != ""
+                    http = NiceHttp.new(config.github.host)
+                    http.headers.authorization = "token #{config.github.token}"
+                    memo.message+="?" unless memo.message.include?('?')
+                    memo.message+="&per_page=50"
+              
+                    resp = http.get("/repos/#{memo.message}")
+                    issues = resp.data.json()
+                    issues = [issues] unless issues.is_a?(Array)
+                    if resp.code == 200
+                      unless issues.empty?
+                        issues.each do |issue|
+                          github_memo = { jira: false, github: true, status: "", memo_id: memo.memo_id, topic: memo.topic, privacy: memo.privacy, user: memo.user, date: memo.date, message: "", type: memo.type }
+                          github_memo.message = issue.title
+                          github_memo.user = issue.user.login
+                          github_memo.date = issue.created_at
+                          if issue.labels.empty?
+                            labels = ''
+                          else
+                            labels = issue.labels.name.sort.join("_").split(" ").join("_") 
+                          end
+                          if memo.topic == :no_topic and !issue.labels.empty?
+                            github_memo.topic = labels
+                          end
+                          case labels
+                          when /bug/i; github_memo.type = ":bug:"
+                          when /docum/i; github_memo.type = ":abc:"
+                          when /task/i; github_memo.type = ":clock1:"
+                          when /enhancem/i, /improvement/i; github_memo.type = ":sunny:"
+                          else github_memo.type = ":memo:"
+                          end
+                          if issue.key?(:events_url)
+                            resp_events = http.get(issue.events_url)
+                            events = resp_events.data.json(:event)
+                            issue.state = "in progress" if events.include?('referenced')
+                          end
+                          case issue.state
+                          when "closed"; github_memo.status = ":heavy_check_mark:"
+                          when "open"; github_memo.status = ":new:"
+                          when "in progress"; github_memo.status = ":runner:"
+                          else github_memo.status = ":heavy_minus_sign:"
+                          end
+                          #todo: check if possible to add link to status instead of github issue
+                          github_memo.status += " <#{issue.html_url}|##{issue.number}>"
+
+                          all_memos[github_memo.topic] ||= []
+                          all_memos[github_memo.topic] << github_memo
+                        end
+                      end
+                    end
+                    http.close
+                  else
+                    memo.jira = false
+                    memo.github = false
+                    all_memos[memo.topic] ||= []
+                    case memo.type
+                    when "memo"; memo.type = ":memo:"
+                    when "note"; memo.type = ":abc:"
+                    when "bug"; memo.type = ":bug:"
+                    when "task"; memo.type = ":clock1:"
+                    when "feature"; memo.type = ":sunny:"
+                    when "issue"; memo.type = ":hammer:"
+                    else memo.type = ":heavy_minus_sign:"
+                    end
+                    all_memos[memo.topic] << memo
+                  end
+                end
+              end
+              message << "   > *_memos_*" unless all_memos.empty?
+
+              if all_memos.key?(:no_topic)
+                all_memos[:no_topic].sort_by { |memo| memo[:date] }.each do |memo|
+                  case memo.privacy
+                  when "private"; priv = " `private`"
+                  when "personal"; priv = " `personal`"
+                  else priv = ""
+                  end
+                  message << "        #{memo.type} #{memo.date.gsub("-", "/")[0..9]}:  #{memo.status} #{memo.message} (#{memo.user} #{memo.memo_id})#{priv}"
+                end
+              end
+              all_memos[:no_topic] = []
+              all_memos.each do |topic, mems|
+                unless mems.empty?
+                  message << "        _`#{topic}`_:"
+                  mems.sort_by { |m| m[:date] }.each do |memo|
+                    case memo.privacy
+                    when "private"; priv = " `private`"
+                    when "personal"; priv = " `personal`"
+                    else priv = ""
+                    end
+                    message << "            #{memo.type} #{memo.date.gsub("-", "/")[0..9]}:  #{memo.status} #{memo.message} (#{memo.user} #{memo.memo_id})#{priv}"
+                  end
+                end
+              end
+            end
+
             unless team.info.empty?
               team.info.split("\n").each do |m|
                 message << ">#{m}"
@@ -236,7 +383,7 @@ class SlackSmartBot
         if team_name.to_s != ""
           message = "\n\n:palm_tree: On vacation / "
           message += ":spiral_calendar_pad: In a meeting / "
-          message += ":face_with_thermometer: Sick leave / "
+          message += ":face_with_thermometer: :baby: Sick leave / "
           message += ":white_circle: Away / "
           message += ":large_yellow_circle: Available in remote timezone / "
           message += ":large_green_circle: Available"
@@ -246,6 +393,9 @@ class SlackSmartBot
         messages.each do |msg|
           respond msg, dest, unfurl_links: false, unfurl_media: false
         end
+        unless team_name.to_s.empty?
+          see_vacations_team(user, team_name, Date.today.strftime("%Y/%m/%d"), add_stats: false)
+        end
       end
     end
   end

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

@@ -0,0 +1,58 @@
+class SlackSmartBot
+  def see_vacations(user, from_user: '', add_stats: true)
+    save_stats(__method__) if add_stats
+
+    get_vacations()
+    
+    from_user_name = ''
+
+    if from_user.empty?
+      from_user_name = user.name
+    else
+      @users = get_users() if @users.empty?
+      user_info = @users.select{|u| u.id == from_user or (u.key?(:enterprise_user) and u.enterprise_user.id == from_user)}[-1]
+      from_user_name = user_info.name
+    end
+
+    if !@vacations.key?(from_user_name) or @vacations[from_user_name].periods.empty?
+      if from_user.empty?
+        respond "You didn't add any time off yet. Use `add vacation from YYYY/MM/DD to YYYY/MM/DD`"
+      else
+        respond "No time off added yet for <@#{from_user}>"
+      end
+    else
+      messages = []
+      messages << "*Time off <@#{from_user}>*" if !from_user.empty?
+      today = Date.today.strftime("%Y/%m/%d")
+      current_added = false
+      past_added = false
+      @vacations[from_user_name].periods.sort_by { |v| v[:from]}.reverse.each do |vac|
+        if !current_added and vac.to >= today
+          messages << "*Current and future periods*" 
+          current_added = true
+        end
+        if !past_added and vac.to < today and from_user.empty?
+          messages << "\n*Past periods*" 
+          past_added = true
+        end
+        unless !from_user.empty? and vac.to < today
+          if !from_user.empty?
+            icon = ":beach_with_umbrella:"
+          elsif vac.type == 'vacation'
+            icon = ':palm_tree:'
+          elsif vac.type == 'sick'
+            icon = ':face_with_thermometer:'
+          elsif vac.type == 'sick child'
+            icon = ':baby:'
+          end
+          if vac.from == vac.to
+            messages << "     #{icon}    #{vac.from}   ##{vac.vacation_id}"
+          else
+            messages << "     #{icon}    #{vac.from} -> #{vac.to}   ##{vac.vacation_id}"
+          end
+        end
+      end
+      respond messages.join("\n")
+    end
+  end
+end

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

@@ -0,0 +1,136 @@
+class SlackSmartBot
+  def see_vacations_team(user, team_name, date, add_stats: true)
+    save_stats(__method__) if add_stats
+
+    get_teams()
+    teams = @teams.deep_copy
+    if teams.empty?
+      respond "There are no teams added yet. Use `add team` command to add a team."
+    elsif team_name.to_s != "" and !teams.key?(team_name.to_sym) and (teams.keys.select { |t| (t.to_s.gsub("-", "").gsub("_", "") == team_name.to_s) }).empty?
+      respond "It seems like the team *#{team_name}* doesn't exist.\nRelated commands `add team TEAM_NAME PROPERTIES`, `see team TEAM_NAME`, `see teams`"
+    else
+      teams.each do |name, team|
+        if team_name==name.to_s or (name.to_s.gsub("-", "").gsub("_", "") == team_name.to_s)
+          team_name = name.to_s
+          break
+        end
+      end
+      date.gsub!('-','/')
+      get_vacations()
+      team = teams[team_name.to_sym]
+      assigned_members = team.members.values.flatten
+      assigned_members.uniq!
+      assigned_members.dup.each do |m|
+        user_info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) or u.name == m or (u.key?(:enterprise_user) and u.enterprise_user.name == m) }[-1]
+        assigned_members.delete(m) if user_info.nil? or user_info.deleted
+      end
+
+      channels_members = []
+      all_team_members = assigned_members.dup
+      if team.channels.key?("members")
+        team_members = []
+        team.channels["members"].each do |ch|
+          get_channels_name_and_id() unless @channels_id.key?(ch)
+          tm = get_channel_members(@channels_id[ch])
+          if tm.nil?
+            respond ":exclamation: Add the Smart Bot to *##{ch}* channel to be able to get the list of members.", dest
+          else
+            channels_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!
+        all_team_members += team_members
+        all_team_members.uniq!
+      end
+      unless all_team_members.empty?
+        blocks_header = 
+          {
+                    "type": "context",
+                    elements: [
+                      {
+                          type: "mrkdwn",
+                          text: "*Time Off #{team_name} team* from #{date} ",
+                        },
+                    ],
+                  }
+        
+        from = Date.parse(date, "%Y/%m/%d")
+        blocks = []
+        all_team_members.each do |m|
+          @users = get_users() if @users.empty?
+          info = @users.select { |u| u.id == m or (u.key?(:enterprise_user) and u.enterprise_user.id == m) or u.name == m or (u.key?(:enterprise_user) and u.enterprise_user.name == m) }[-1]
+          unless info.nil?
+            info = get_user_info(info.id)
+            if @vacations.key?(m)
+              v = ""
+              (from..(from+20)).each do |d|
+                v+="#{d.strftime("%d")} " if d.wday==1 or d==from
+                on_vacation = false
+                @vacations[m].periods.each do |p|
+                  if p.from <= d.strftime("%Y/%m/%d") and p.to >= d.strftime("%Y/%m/%d")
+                    if d.wday == 0 or d.wday == 6
+                      v+=":large_orange_square: "
+                    else
+                      v+=":large_red_square: "
+                    end
+                    on_vacation=true
+                    break
+                  end 
+                end
+                unless on_vacation
+                  if d.wday == 0 or d.wday == 6
+                    v += ":large_yellow_square: "
+                  else
+                    v+= ":white_square: " 
+                  end
+                end
+              end
+            else
+              v = ""
+              (from..(from+20)).each do |d|
+                if d.wday==1 or d==from
+                  v += "#{d.strftime("%d")} " 
+                end
+                if d.wday == 0 or d.wday == 6
+                  v += ":large_yellow_square: "
+                else
+                  v += ":white_square: "
+                end
+              end
+            end
+
+            blocks << {
+              type: "context",
+              elements: [
+                {
+                            type: "image",
+                            image_url: info.user.profile.image_24,
+                            alt_text: info.user.name,
+                          },
+                          {
+                            type: "plain_text",
+                            text: v
+                          }  
+              ],
+            }
+          end
+        end
+        first = true
+        blocks.each_slice(10).each do |b|
+          if first 
+            b.unshift(blocks_header)
+            first = false
+          end
+          respond blocks: b
+        end
+
+      end
+
+
+    end
+  end
+end

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

@@ -0,0 +1,58 @@
+class SlackSmartBot
+  def set_memo_status(user, team_name, memo_id, status)
+    save_stats(__method__) if answer.empty?
+
+    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 + config.masters).flatten.include?(user.name)
+      respond "You have to be a member of the team or a Master admin to be able to set the status of a memo."
+    elsif !@teams[team_name.to_sym].key?(:memos) or @teams[team_name.to_sym][:memos].empty? or !@teams[team_name.to_sym][:memos].memo_id.include?(memo_id.to_i)
+      respond "It seems like there is no memo with id #{memo_id}"
+    elsif @teams[team_name.to_sym][:memos].memo_id.include?(memo_id.to_i)
+      memo_selected = @teams[team_name.to_sym][:memos].select { |m| m.memo_id == memo_id.to_i }[-1]
+      if memo_selected.type == 'jira' or memo_selected.type == 'github'
+        #todo: add tests for jira and github
+        respond "The memo specified is a #{memo_selected.type} and the status in those cases are linked to the specific issues in #{memo_selected.type}."
+      elsif memo_selected.privacy == "personal" and memo_selected.user != user.name
+        respond "Only the creator can set the status of a personal memo."
+      else
+        answer_delete
+        memos = []
+        message = ""
+        get_teams()
+        @teams[team_name.to_sym][:memos].each do |memo|
+          if memo.memo_id == memo_id.to_i
+            memo.status = status
+            message = memo.message
+          end
+          memos << memo
+        end
+        @teams[team_name.to_sym][:memos] = memos
+        update_teams()
+        respond "The memo status has been updated on team #{team_name}: #{message}"
+      end
+    end
+  end
+end

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

@@ -2,9 +2,29 @@ class SlackSmartBot
   def update_team(user, team_name, new_name: "", new_info: "", delete_opts: "", add_opts: "")
     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
     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 !(@teams[team_name.to_sym].members.values + [@teams[team_name.to_sym].creator] + config.masters).flatten.include?(user.name)
+    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 update this team."
     else
       wrong = false
@@ -69,6 +89,7 @@ class SlackSmartBot
             last_type = opt
           elsif opt.match(/<@(\w+)>/i)
             member_id = $1
+            last_type = 'no_type' if last_type.nil?
             member_info = @users.select { |u| u.id == member_id or (u.key?(:enterprise_user) and u.enterprise_user.id == member_id) }[-1]
             @teams[team_name.to_sym].members[last_type] ||= []
             @teams[team_name.to_sym].members[last_type] << member_info.name