updated changes according to comments from ben

demansoltech · Jul 8, 2021 · 2d4c6bc · 2d4c6bc
1 parent bcd11f0
commit 2d4c6bc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/server/hooks.go b/server/hooks.go
@@ -18,10 +18,13 @@ func (p *Plugin) UserHasJoinedTeam(c *plugin.Context, teamMember *model.TeamMemb
 	}
 
 	for _, message := range p.getWelcomeMessages() {
-		if message.TeamName == data.Team.Name {
+		switch message.TeamName {
+		case data.Team.Name:
 			go p.processWelcomeMessage(*data, *message)
-		} else if message.TeamName == "*" {
+		case "*":
 			go p.processWelcomeMessage(*data, *message)
+		default:
+			p.API.LogError("Couldn't find the message for the team")
 		}
 	}
 }

diff --git a/server/welcomebot.go b/server/welcomebot.go
@@ -66,7 +66,7 @@ func (p *Plugin) newSampleMessageTemplate(teamName, userID string) (*MessageTemp
 		return nil, fmt.Errorf("failed to query user %s: %w", userID, err)
 	}
 
-	if data.Team, err = p.API.GetTeamByName(strings.ToLower(teamName)); err != nil {
+	if data.Team, err = p.API.GetTeamByName(teamName); err != nil {
 		p.API.LogError("failed to query team", "team_name", teamName, "err", err)
 		return nil, fmt.Errorf("failed to query team %s: %w", teamName, err)
 	}