Skip to content

MM-64925 - prevent slack import email auto validation for non admin users #8284

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Aug 11, 2025

Conversation

pvev
Copy link
Contributor

@pvev pvev commented Aug 10, 2025

Summary

This PR clarifies how the email validation will work during the slack import process making clear that for imports performed by system admins the imported user emails will be automatically validated and users will be able to reset password directly as opposite than migrations performed by non-admins users where imported users will have to validate their emails first before being able to reset the password for getting into the workspace.

Ticket Link

mattermost/mattermost#33609

Copy link

Newest code from mattermost has been published to preview environment for Git SHA 27bc5a9

Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @pvev!

@cwarnermm cwarnermm added the 3: Reviews Complete All reviewers have approved the pull request label Aug 11, 2025
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 28bfd13

Copy link

Newest code from mattermost has been published to preview environment for Git SHA 79dc636

Copy link

Newest code from mattermost has been published to preview environment for Git SHA d95c56f

Copy link

Newest code from mattermost has been published to preview environment for Git SHA bc16079

@cwarnermm cwarnermm merged commit c04d66f into master Aug 11, 2025
5 checks passed
@cwarnermm cwarnermm deleted the MM-64925-slack-import-email-validation branch August 11, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants