[MM-46644] Auto respond message

[vish9812]

Summary

Fix auto-respond message

Ticket Link

JIRA - https://mattermost.atlassian.net/browse/MM-46644

Release Note

NONE

[ashishbhate]

We should add a unit test

[mattermod]

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

[srkgupta]

I see that the validation is in place only while creating a user & importing a user. Can we also add this check in other places, for example:

func (us SqlUserStore) Update(user *model.User, trustedUpdateData bool) (*model.UserUpdate, error) {

func (us SqlUserStore) 
(userID string, props map[string]string) error {

[vish9812]

@srkgupta Validation is already added in the funcs, you have mentioned.

• Update
• UpdateNotifyProps

@ashishbhate Added tests and logging

[srkgupta]

Additional changes requested. Please check the channel for more details.

[vish9812]

/update-branch

[agnivade]

Lovely!

[agnivade]

/e2e-test

[mattermod]

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/240843

[vish9812]

@srkgupta @ashishbhate It's up for review.

[mattermod]

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

[mattermod]

Trying to auto merge this PR.

[mattermod]

Pull Request successfully merged
SHA: 6e9b808

[mattermod]

Cherry pick is scheduled.

[mattermod]

Error trying doing the automated Cherry picking. Please do this manually

+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '140.82.113.4' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-server
   da03fbb05..f6c7d99a2  MM-43850_separate_leave_team_events -> upstream/MM-43850_separate_leave_team_events
   9cf76e319..018684039  MM-45066-move-app-methods-to-the-context-logger -> upstream/MM-45066-move-app-methods-to-the-context-logger
   168151600..9a11b694b  MM-46275_production-mpa -> upstream/MM-46275_production-mpa
   138edf0d6..5b0051221  MM-47228-restrict-guest-invitation-flow -> upstream/MM-47228-restrict-guest-invitation-flow
   70c74fc10..b68461f71  cloud        -> upstream/cloud
   040087677..59eb063d9  lev-Scopes-2 -> upstream/lev-Scopes-2
   5a4cac43a..6e9b808ef  master       -> upstream/master
   43cbd0333..dbf5cb24a  release-7.4  -> upstream/release-7.4
Fetching origin
Failed to add the RSA host key for IP address '140.82.114.3' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.4-1665147734
Switched to a new branch 'automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.4-1665147734'
Branch 'automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.4-1665147734' set up to track remote branch 'release-7.4' from 'upstream'.

+++ About to attempt cherry pick of PR #20900 with merge commit 6e9b808efd7f6ded8d4de20bac98e087d82e0252.

error: could not apply 6e9b808ef... [MM-46644] Auto respond message (#20900)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU store/storetest/user_store.go
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

[mm-cloud-bot]

Test server destroyed

[amyblais]

/cherry-pick cloud

[mattermod]

Cherry pick is scheduled.

[amyblais]

/cherry-pick release-7.3

[mattermod]

Cherry pick is scheduled.

[mattermod]

Error trying doing the automated Cherry picking. Please do this manually

+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '140.82.114.3' to the list of known hosts (/app/.ssh/known_hosts).
Fetching origin
Failed to add the RSA host key for IP address '140.82.114.3' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.3-1665147920
Switched to a new branch 'automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.3-1665147920'
Branch 'automated-cherry-pick-of-mattermost-server-#20900-upstream-release-7.3-1665147920' set up to track remote branch 'release-7.3' from 'upstream'.

+++ About to attempt cherry pick of PR #20900 with merge commit 6e9b808efd7f6ded8d4de20bac98e087d82e0252.

error: could not apply 6e9b808ef... [MM-46644] Auto respond message (#20900)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

+++ Conflicts detected:

UU model/utils.go
UU store/storetest/user_store.go
Aborting.

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.