mattermost / mattermost-developer-documentation

Mattermost developer documentation.
https://developers.mattermost.com
BSD 3-Clause "New" or "Revised" License
79 stars 383 forks source link

Update developer-setup.md to provide no-docker setup #1351

Closed sinansonmez closed 3 months ago

sinansonmez commented 3 months ago

Summary

Update developer-setup.md to provide no-docker setup information

mattermost-build commented 3 months ago

Hello @sinansonmez,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

sinansonmez commented 3 months ago

When you make those changes, could you move those steps to a separate section at the bottom of the doc and change that "Install Docker" step to point the user to the new section if they don't want to use Docker? I think we should split those steps into their own section because I think people will miss that those steps are optional and go through them after installing Docker. I want to keep the main instructions simpler because I screw up complicated instructions all the time 😅

@hmhealey @cwarnermm Thanks for the feedback. Unfortunately, it is not clear to me how to proceed.

Currently there are 3 main sections

Under Setup the Mattermost Server, there are 13 steps. Should I create a new main section or add the 14th step under Setup the Mattermost Server ?

cwarnermm commented 3 months ago

Hi @sinansonmez! The ask is to separate the Docker steps within their own section (at the top of the page as the primary way forward) from the not-using-Docker steps you're providing. @hmhealey is suggesting we move the non-Docker info to its own section elsewhere on the page to avoid reader confusion.

The reader's experience would look like this:

Meanwhile, following the Setup the Mattermost Server section, there's a new section about setting up Mattermost Server without Docker. That's where your steps would live.

hmhealey commented 3 months ago

Yep, that's right. So with your steps, there'd be 4 sections:

sinansonmez commented 3 months ago

thanks for the explanation @hmhealey @cwarnermm the page is updated as requested

github-actions[bot] commented 3 months ago

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

github-actions[bot] commented 3 months ago

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

github-actions[bot] commented 3 months ago

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