-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
UPDATE: add latest update section & Cloudflare instruction #14719
Conversation
This pull request has been mentioned on Discourse Meta. There might be relevant details there: https://meta.discourse.org/t/missing-a-vital-step-in-the-official-install-guide/207106/5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small typo
Updates are already covered by discourse/docs/INSTALL-cloud.md Line 119 in 1c7833b
|
The security updates are only covered after the install. There are some dependency that needs to be updated before discourse install. This person was having an issue installing discourse until he upgraded. https://meta.discourse.org/t/problem-with-self-hosted-instance/207088 |
I don't think we should do a full section on this because the guide is already pretty long. @xfalcox what do you think about folding a sentence into the previous section instead? Your call, we could also leave it as it is for now but I think reminding people to update their packages is a good idea anyway. |
Can you give me the text so that I can update my PR please |
Sorry but I am closing this, let's discuss this documentation change on meta. |
install-cloud.md file update