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

Docker Documentation #58

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Docker Documentation #58

wants to merge 6 commits into from

Conversation

Casraw
Copy link

@Casraw Casraw commented Mar 7, 2023

Here are the docs for Docker


`export POSTGRES_USER=postgres` <br />
`export POSTGRES_PASSWORD=postgres` <br />
`export POSTGRES_DATABASE=fosscord`
Copy link
Member

@MaddyUnderStars MaddyUnderStars Mar 7, 2023

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

Copy link
Author

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 Show resolved Hide resolved
Comment on lines 21 to 26
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"
```
Copy link
Member

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?

Copy link
Author

Choose a reason for hiding this comment

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

Insert better description

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
Copy link
Member

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.

Copy link
Author

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

Comment on lines 50 to 55
## Docker

To use docker please read the following page:

- [Docker](docker.md)

Copy link
Member

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

Copy link
Author

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

@Casraw
Copy link
Author

Casraw commented Apr 1, 2023

I will rewrite this too, because there is still fosscord in it. Will use the readme from the repo for it.

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.

3 participants