Skip to content

update slack migration docs#8741

Open
vish9812 wants to merge 3 commits intomasterfrom
slack
Open

update slack migration docs#8741
vish9812 wants to merge 3 commits intomasterfrom
slack

Conversation

@vish9812
Copy link
Contributor

@vish9812 vish9812 commented Feb 13, 2026

  • Updated Slack export terminology from "Regular export"/"Corporate export" to current Slack terms ("Public channels export"/"All channels and conversations export") and corrected the UI navigation path
  • Removed the slack-advanced-exporter section and all references — Slack exports already include emails and file download links, making this tool unnecessary
  • Removed the Additional Tools subsection (mm-emoji, mmimportjs, slack-migrate-pinned-posts) as they are archived, broken, or no longer maintained
  • Added mmetl check slack validation step before running the transform
  • Added documentation for useful mmetl transform flags: --skip-attachments, --allow-download, --default-email-domain, --skip-empty-emails, --discard-invalid-props
  • Added mmctl import validate step before processing the import
  • Fixed incorrect command mmctl import list process to mmctl import job list
  • Fixed typos and minor errors throughout the document

PS: I've not tested it with the Enterprise export. So, no changes around that.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Slack-to-Mattermost migration documentation to improve accuracy, clarity, and alignment with current tooling. The updates remove references to the deprecated slack_advanced_exporter tool, add a validation step using mmetl check slack, correct terminology around Slack export types, fix numerous spelling and grammar errors, and refine configuration guidance.

Changes:

  • Removed deprecated slack_advanced_exporter tool instructions and streamlined to focus on the officially supported mmetl tool
  • Added validation step for Slack exports before transformation using mmetl check slack
  • Corrected multiple spelling and grammar errors throughout the document (necessary, enough, additional, associated, Starred, Threaded conversations)
  • Updated Slack export type terminology to "Public channels export" and "All channels and conversations export" with clarified descriptions
  • Fixed configuration guidance for TeamSettings.MaxChannelsPerTeam and removed extra space in ElasticsearchSettings.EnableIndexing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Contributor

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

1 similar comment
@github-actions
Copy link
Contributor

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


commit;

Additional tools
Copy link
Member

Choose a reason for hiding this comment

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

Are these tools no longer needed? If so, can we add a line explaining why?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's mentioned in the PR description: 🙂

  • Removed the Additional Tools subsection (mm-emoji, mmimportjs, slack-migrate-pinned-posts) as they are archived, broken, or no longer maintained

Copy link
Member

Choose a reason for hiding this comment

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

I mean explaining to the reader why these tools they've used in the past are no longer needed.

@cwarnermm cwarnermm requested review from Combs7th and removed request for cwarnermm February 17, 2026 20:15
@cwarnermm cwarnermm added 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor Guidance labels Feb 17, 2026
@github-actions
Copy link
Contributor

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

@vish9812 vish9812 requested a review from wiggin77 February 20, 2026 09:33
@esethna
Copy link
Contributor

esethna commented Mar 2, 2026

@wiggin77 can you help with a re-review on this as needed?

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

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

Copy link
Member

@wiggin77 wiggin77 left a comment

Choose a reason for hiding this comment

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

LGTM 👍 @fmartingr should also have a look before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor Guidance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants