[MI-3395] Fixed the issue of email notifications being sent for synth…

…etic users (#240) Added the logic of setting notify_props of the user when he's being created
mattermost · Aug 14, 2023 · 2cd5de6 · 2cd5de6
1 parent 254fbb9
commit 2cd5de6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/server/handlers/getters.go b/server/handlers/getters.go
@@ -130,6 +130,8 @@ func (ah *ActivityHandler) getOrCreateSyntheticUser(user *msteams.User, createSy
 			Password:  ah.plugin.GenerateRandomPassword(),
 			RemoteId:  &shortUserID,
 		}
+		newMMUser.SetDefaultNotifications()
+		newMMUser.NotifyProps[model.EmailNotifyProp] = "false"
 
 		userSuffixID := 1
 		for {

diff --git a/server/plugin.go b/server/plugin.go
@@ -501,6 +501,8 @@ func (p *Plugin) syncUsers() {
 				FirstName: msUser.DisplayName,
 				Username:  username,
 			}
+			newMMUser.SetDefaultNotifications()
+			newMMUser.NotifyProps[model.EmailNotifyProp] = "false"
 
 			var newUser *model.User
 			for {