Skip to content
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

Add info to slack migration docs #6788

Merged
merged 24 commits into from
Dec 1, 2023
Merged

Conversation

mickmister
Copy link
Contributor

Summary

This PR fills in information in the Slack migration docs to reflect findings in my experience with migrating from Slack to Mattermost

Ticket Link

Copy link

Newest code from mickmister has been published to preview environment for Git SHA 440ae66

source/onboard/migrating-to-mattermost.rst Outdated Show resolved Hide resolved
source/onboard/migrating-to-mattermost.rst Outdated Show resolved Hide resolved
source/onboard/migrating-to-mattermost.rst Outdated Show resolved Hide resolved
source/onboard/migrating-to-mattermost.rst Outdated Show resolved Hide resolved
@cwarnermm cwarnermm added the 2: Editor Review Requires review by an editor label Nov 15, 2023
@cwarnermm
Copy link
Member

Thank you, @mickmister! Let me know how best I can support you with this lift, and please feel free to reorder the content on the page based on your initial feedback.

@cwarnermm cwarnermm added the Work In Progress Not yet ready for review label Nov 15, 2023
Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 9aa2cb8

@mickmister
Copy link
Contributor Author

@cwarnermm I think doing a side-by-side comparison of the two versions of the rendered docs would be helpful. The markdown diff makes it hard to inspect the new changes, and seeing the new version without the context of the old will be difficult to know what the changes are

Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA cd69b48

@cwarnermm
Copy link
Member

@mickmister - I've reorganized the content on the page based on your excellent feedback in my commit. Also adjusted minor formatting to align with the documentation style guide.

Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA 0a9747a

Copy link

Newest code from cwarnermm has been published to preview environment for Git SHA b2186c4

@mickmister mickmister requested review from cwarnermm and removed request for cwarnermm November 21, 2023 09:38
Copy link

Newest code from mickmister has been published to preview environment for Git SHA 954eb44

@mickmister
Copy link
Contributor Author

@cwarnermm FYI I created an issue to remove the marketing verbiage around bespoke #6816

Copy link

Newest code from mickmister has been published to preview environment for Git SHA 8f52948

Copy link

Newest code from mickmister has been published to preview environment for Git SHA 1fb5e13

Copy link

Newest code from mickmister has been published to preview environment for Git SHA e66ffc0

Copy link

Newest code from mickmister has been published to preview environment for Git SHA 4ab658c

Copy link
Contributor

@sadohert sadohert left a comment

Choose a reason for hiding this comment

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

@mickmister fantastic job man! I found this one to be a gnarly one to lay out... you have a nice flow through that incorporates all the related tools (except maybe one... see below).

One case I'm wondering about is when a customer is merging multiple workspaces, and what should our guidance be here... do we add considerations for this right on the same page, or link of somewhere else as an "Advanced Topic". This is a gnarly topic itself, and introduces the need for the sync tool you built to help address user merge issues.

source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Show resolved Hide resolved
8. Grant the app permissions when prompted.
9. Copy the **Bot User OAuth Token** and save it somewhere convenient.

We'll now use **Bot User OAuth Token** with the ``slack-advanced-exporter`` tool to download emails and attachments. Download the latest release of ``slack-advanced-exporter`` for your OS and architecture `here <https://github.com/grundleborg/slack-advanced-exporter/releases/>`__ and extract the executable from the download.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should move it to the MM org and commit to owning issues with it.

source/onboard/migrate-from-slack.rst Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
source/onboard/migrate-from-slack.rst Show resolved Hide resolved
@mickmister mickmister dismissed fmartingr’s stale review December 1, 2023 17:29

I'm addressing the remaining screenshot request from Felipe. Wanting to get this merged today so dismissing his review

Copy link
Contributor

@fmartingr fmartingr left a comment

Choose a reason for hiding this comment

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

👍 LGTM

Copy link

github-actions bot commented Dec 1, 2023

Newest code from mickmister has been published to preview environment for Git SHA fbc46d6

Copy link

github-actions bot commented Dec 1, 2023

Newest code from mickmister has been published to preview environment for Git SHA 352b601

Copy link

github-actions bot commented Dec 1, 2023

Newest code from mickmister has been published to preview environment for Git SHA acfe878

@mickmister
Copy link
Contributor Author

I reverted adding the screenshots, as that is specific to only exporting public channels from Slack. I think customers will mostly be doing the full export, which those screenshots don't help with, so I think they will more of a distraction than anything, especially because of how big the images are rendered on the docs page

Copy link

github-actions bot commented Dec 1, 2023

Newest code from mickmister has been published to preview environment for Git SHA 9224dea

@mickmister
Copy link
Contributor Author

Copy link
Contributor

@sadohert sadohert left a comment

Choose a reason for hiding this comment

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

LGTM. Nice one @mickmister !

source/onboard/migrate-from-slack.rst Outdated Show resolved Hide resolved
Co-authored-by: Stuart Doherty <stu.doherty@gmail.com>
Copy link

github-actions bot commented Dec 1, 2023

Newest code from mickmister has been published to preview environment for Git SHA a520ffa

@mickmister mickmister merged commit a6fb979 into master Dec 1, 2023
4 checks passed
@mickmister mickmister deleted the slack-migration-docs-michaelk branch December 1, 2023 20:08
@mattermost-build mattermost-build removed the Work In Progress Not yet ready for review label Dec 1, 2023
@amyblais amyblais removed the 2: Editor Review Requires review by an editor label Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants