Conversation
There was a problem hiding this comment.
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_exportertool instructions and streamlined to focus on the officially supportedmmetltool - 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.MaxChannelsPerTeamand removed extra space inElasticsearchSettings.EnableIndexing
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Newest code from mattermost has been published to preview environment for Git SHA 92255d6 |
1 similar comment
|
Newest code from mattermost has been published to preview environment for Git SHA 92255d6 |
|
|
||
| commit; | ||
|
|
||
| Additional tools |
There was a problem hiding this comment.
Are these tools no longer needed? If so, can we add a line explaining why?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
I mean explaining to the reader why these tools they've used in the past are no longer needed.
|
Newest code from mattermost has been published to preview environment for Git SHA c88dbd1 |
|
@wiggin77 can you help with a re-review on this as needed? |
|
Newest code from mattermost has been published to preview environment for Git SHA 4d21d1a |
wiggin77
left a comment
There was a problem hiding this comment.
LGTM 👍 @fmartingr should also have a look before merging.
PS: I've not tested it with the Enterprise export. So, no changes around that.