search

mattermost / mattermost-plugin-msteams

MS Teams plugin for Mattermost
Other
13 stars 11 forks source link

MM-57563 - update method to allow self posts to sync, only if connected #583

Closed sbishel closed 4 months ago

sbishel commented 5 months ago

Summary

Changes how we make sure self DMs sync, rather then setting the variable chatMembersSpanPlatforms. Skip the "Selective Sync" check altogether.

Most of these changes are due to the change of function name from ChatSpansPlatforms to ChatShouldSync. The functions that called it actually set variable names of shouldSync and with the check for single user, makes more sense.

Ticket Link

Fixes https://mattermost.atlassian.net/browse/MM-57563

sbishel commented 5 months ago

I need to add a couple unit tests...but ready to take a look if we want to try for v1.11.0.

wiggin77 commented 4 months ago

/update-branch

sbishel commented 4 months ago

will qa in bug bash.