-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Docker Documentation #58
base: master
Are you sure you want to change the base?
Conversation
docs/setup/server/docker.md
Outdated
|
||
`export POSTGRES_USER=postgres` <br /> | ||
`export POSTGRES_PASSWORD=postgres` <br /> | ||
`export POSTGRES_DATABASE=fosscord` |
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.
you don't need to use HTML here. just use a normal code block instead of inline code.
like this
```
like this
```
alternatively, just put them in a table. you should also put them in the env vars docs
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.
Yes i did it
docs/setup/server/docker.md
Outdated
At first start, you get an | ||
|
||
``` | ||
db_1 | 2023-03-04 17:28:25.790 UTC [63] ERROR: relation "config" does not exist at character 31 | ||
db_1 | 2023-03-04 17:28:25.790 UTC [63] STATEMENT: SELECT COUNT(1) AS "cnt" FROM "config" "ConfigEntity" | ||
``` |
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.
why is this mentioned? should they do something about it? restart the server?
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.
Insert better description
docs/setup/server/docker.md
Outdated
db_1 | 2023-03-04 17:28:25.790 UTC [63] STATEMENT: SELECT COUNT(1) AS "cnt" FROM "config" "ConfigEntity" | ||
``` | ||
|
||
## prod environment with S3 file storage |
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.
this section could probably just be merged with the above one instead. the content is basically the same, minus the env vars. if you go with an env vars table, just add these to it.
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.
Merged and insert into env variables
docs/setup/server/index.md
Outdated
## Docker | ||
|
||
To use docker please read the following page: | ||
|
||
- [Docker](docker.md) | ||
|
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.
should probably be mentioned somewhere at the top instead
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.
ok it is on top now
I will rewrite this too, because there is still fosscord in it. Will use the readme from the repo for it. |
Here are the docs for Docker