docs: Sweepfixes II Electric Boogaloo #2083

Merged
Aranjedeath merged 17 commits from stratself/continuwuity:stratself/docs-sweepfixes-ii into main 2026-08-05 18:00:11 +00:00
Member

Small changes to various pages of docs to improve wording/clarity/linkage/etc and update some information. https://muc.muoi.me for preview as usual.

Pull request checklist:

  • This pull request targets the main branch, and the branch is named something other than
    main.
  • I have written an appropriate pull request title and my description is clear.
  • I understand I am responsible for the contents of this pull request.
  • I have followed the contributing guidelines:
<!-- In order to help reviewers know what your pull request does at a glance, you should ensure that 1. Your PR title is a short, single sentence describing what you changed 2. You have described in more detail what you have changed, why you have changed it, what the intended effect is, and why you think this will be beneficial to the project. If you have made any potentially strange/questionable design choices, but didn't feel they'd benefit from code comments, please don't mention them here - after opening your pull request, go to "files changed", and click on the "+" symbol in the line number gutter, and attach comments to the lines that you think would benefit from some clarification. --> Small changes to various pages of docs to improve wording/clarity/linkage/etc and update some information. https://muc.muoi.me for preview as usual. <!-- Example: This pull request allows us to warp through time and space ten times faster than before by double-inverting the warp drive with hyperheated jump fluid, both making the drive faster and more efficient. This resolves the common issue where we have to wait more than 10 milliseconds to engage, use, and disengage the warp drive when travelling between galaxies. --> <!-- Closes: #... --> <!-- Fixes: #... --> <!-- Uncomment the above line(s) if your pull request fixes an issue or closes another pull request by superseding it. Replace `#...` with the issue/pr number, such as `#123`. --> **Pull request checklist:** <!-- You need to complete these before your PR can be considered. If you aren't sure about some, feel free to ask for clarification in #dev:continuwuity.org. --> - [x] This pull request targets the `main` branch, and the branch is named something other than `main`. - [x] I have written an appropriate pull request title and my description is clear. - [x] I understand I am responsible for the contents of this pull request. - I have followed the [contributing guidelines][c1]: - [x] My contribution follows the [code style][c2], if applicable. - [x] I ran [pre-commit checks][c1pc] before opening/drafting this pull request. - [x] I have [tested my contribution][c1t] (or proof-read it for documentation-only changes) myself, if applicable. This includes ensuring code compiles. - [x] My commit messages follow the [commit message format][c1cm] and are descriptive. <!-- Notes on these requirements: - While not required, we encourage you to sign your commits with GPG or SSH to attest the authenticity of your changes. - While we allow LLM-assisted contributions, we do not appreciate contributions that are low quality, which is typical of machine-generated contributions that have not had a lot of love and care from a human. Please do not open a PR if all you have done is asked ChatGPT to tidy up the codebase with a +-100,000 diff. - In the case of code style violations, reviewers may leave review comments/change requests indicating what the ideal change would look like. For example, a reviewer may suggest you lower a log level, or use `match` instead of `if/else` etc. - In the case of code style violations, pre-commit check failures, minor things like typos/spelling errors, and in some cases commit format violations, reviewers may modify your branch directly, typically by making changes and adding a commit. Particularly in the latter case, a reviewer may rebase your commits to squash "spammy" ones (like "fix", "fix", "actually fix"), and reword commit messages that don't satisfy the format. - Pull requests MUST pass the `Checks` CI workflows to be capable of being merged. This can only be bypassed in exceptional circumstances. If your CI flakes, let us know in matrix:r/dev:continuwuity.org. - Pull requests have to be based on the latest `main` commit before being merged. If the main branch changes while you're making your changes, you should make sure you rebase on main before opening a PR. Your branch will be rebased on main before it is merged if it has fallen behind. - We typically only do fast-forward merges, so your entire commit log will be included. Once in main, it's difficult to get out cleanly, so put on your best dress, smile for the cameras! --> [c1]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md [c2]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/docs/development/code_style.mdx [c1pc]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#pre-commit-checks [c1t]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#running-tests-locally [c1cm]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#commit-messages
stratself force-pushed stratself/docs-sweepfixes-ii from e07b43ee96
Some checks failed
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Auto Labeler / Apply labels based on changed files (pull_request_target) Successful in 4s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Check changed files (pull_request) Successful in 4s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Failing after 2m6s
to 31b1402aa2
Some checks failed
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Check changed files (pull_request) Successful in 6s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Failing after 1m7s
2026-08-01 08:01:28 +00:00
Compare
stratself force-pushed stratself/docs-sweepfixes-ii from 31b1402aa2
Some checks failed
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Check changed files (pull_request) Successful in 6s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Failing after 1m7s
to fc614e1931
All checks were successful
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 4s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m1s
2026-08-01 08:05:37 +00:00
Compare
stratself force-pushed stratself/docs-sweepfixes-ii from fc614e1931
All checks were successful
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 4s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m1s
to 6cc03c154a
All checks were successful
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 16s
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 7s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m12s
Update flake hashes / update-flake-hashes (pull_request) Successful in 1m19s
2026-08-04 15:53:05 +00:00
Compare
@ -55,3 +93,3 @@
### For IPv4-only users
If you don't have IPv6 connectivity, changing `ip_lookup_strategy` to only resolve for IPv4 will reduce unnecessary AAAA queries.
If you don't have IPv6 connectivity, change `ip_lookup_strategy` to `1`, in order to only resolve for IPv4 and remove unnecessary AAAA queries.
Owner

where? ;)

where? ;)
stratself marked this conversation as resolved
@ -74,11 +112,11 @@ Some values that are commonly tuned include:
- Increase `discard-timeout` to something like `4800` to wait longer for upstream resolvers, as recursion can take a long time to respond to some domains. Continuwuity default to `dns_timeout = 10` seconds, so dropping requests early would lead to unnecessary retries and/or failures.
Owner

s/default/defaults/

s/default/defaults/
stratself marked this conversation as resolved
@ -130,6 +143,6 @@ management.
Built-in S3 support is also planned, but for now using a "S3 filesystem" on
Owner

Might want to update this as well, we don't plan an S3 backend (verbiage likely from conduwuit era), but do recommend the "S3 Filesystem" solutions if somebody needs that.

Might want to update this as well, we don't plan an S3 backend (verbiage likely from conduwuit era), but do recommend the "S3 Filesystem" solutions if somebody needs that.
stratself marked this conversation as resolved
Aranjedeath approved these changes 2026-08-04 19:29:53 +00:00
Dismissed
@ -131,3 +143,1 @@
`media/` works. Continuwuity also sends a `Cache-Control` header of 1 year and
immutable for all media requests (download and thumbnail) to reduce unnecessary
media requests from browsers, reduce bandwidth usage, and reduce load.
While Continuwuity does not implement built-in S3 support, using an "S3 filesystem
Member

Missing ending quote?

Missing ending quote?
Member

I would argue there shouldn't be quotes here at all. "an S3 filesystem mount" would be preferred

I would argue there shouldn't be quotes here at all. "an S3 filesystem mount" would be preferred
Member

Agreed

Agreed
stratself marked this conversation as resolved
@ -54,0 +58,4 @@
### Docker DNS
If you follow the [DNS tuning guide](./guides/dns.mdx#for-docker-users) and mount a custom `/etc/resolv.conf`, you may not be able to connect to appservices containers via hostname. Follow the workaround [there](./guides/dns.mdx#for-docker-users) to connect to your appservice via static IPs.
Member

"appservices' containers using their hostnames. Follow the workaround in the guide to connect to your appservice using a static IP."

"appservices' containers using their hostnames. Follow the workaround in the guide to connect to your appservice using a static IP."
stratself marked this conversation as resolved
@ -65,3 +64,3 @@
:::note Docker DNS Performance
Docker's default DNS resolver are known to [cause timeout issues](../troubleshooting#dns-issues) for Matrix federation. To bypass it and use a more performant resolver, mount a custom `/etc/resolv.conf` config file into the Continuwuity container.
Docker's default DNS resolver are known to [**cause timeout issues**](../troubleshooting#dns-issues) for Matrix federation. To bypass it and use a more performant resolver, we will mount a custom `/etc/resolv.conf` config file into the Continuwuity container.
Member

"is known to", and I'm not sure why this is bold. Also, the previous version without "we will" was stronger.

"is known to", and I'm not sure why this is bold. Also, the previous version without "we will" was stronger.
Member

+1 here, phrasing without "we will" also follows the grammar used for the rest of the document. I would also remove the "we will be using" on line 39

+1 here, phrasing without "we will" also follows the grammar used for the rest of the document. I would also remove the "we will be using" on line 39
Author
Member

It is bold because it is quite important.

"We will" is there to say that the example configs will mount the file. Better wording suggestions are welcome.

It is bold because it is quite important. "We will" is there to say that the example configs will mount the file. Better wording suggestions are welcome.
First-time contributor
  • I feel that it reads better without "we will". It's quite clear from context when rendered that it's referring to the config snippets immediately following, although the stop at the end of the sentence could be replaced with a colon.
  • "are known to" -> "is known to"
  • In combination with the link formatting, and since it's all inside a box anyway, I don't think the bold text makes much of a difference. If removed, though, it should also be removed from "DNS tuning guide (recommended)" later in the box.
- I feel that it reads better without "we will". It's quite clear from context when rendered that it's referring to the config snippets immediately following, although the stop at the end of the sentence could be replaced with a colon. - "are known to" -> "is known to" - In combination with the link formatting, and since it's all inside a box anyway, I don't think the bold text makes much of a difference. If removed, though, it should also be removed from "DNS tuning guide (recommended)" later in the box.
Author
Member

Fixed "we will", "is known to" and updated preview on muc.muoi.me

Fixed "we will", "is known to" and updated preview on muc.muoi.me
stratself marked this conversation as resolved
@ -90,0 +90,4 @@
Create a Caddyfile with the following content:
```
https://example.com:443, https://example.com:8448 {
Member

https:// and :443 need not be specified (unless it does need to be with groups of other domains but I truly do not think so)

https:// and :443 need not be specified (unless it does need to be with groups of other domains but I truly do not think so)
stratself marked this conversation as resolved
@ -187,6 +197,21 @@ Test that your setup works by following these [instructions](./generic.mdx#how-d
Check your container logs using `docker-compose logs --follow` to debug any issues. See the [Troubleshooting](../troubleshooting.mdx) page for common errors and how to fix them.
## Updating
Member

Is a docker compose tutorial really necessary?

Is a docker compose tutorial really necessary?
Member

A short guide like this does no harm. I would however, if it is to stay, replace docker-compose with docker compose, as docker-compose was deprecated some years ago

A short guide like this does no harm. I would however, if it is to stay, replace `docker-compose` with `docker compose`, as `docker-compose` was [deprecated some years ago](https://www.docker.com/blog/new-docker-compose-v2-and-v1-deprecation/)
Author
Member

Replacing all instances of docker-compose with docker compose

Replacing all instances of `docker-compose` with `docker compose`
stratself marked this conversation as resolved
@ -55,3 +55,3 @@
- `/.well-known/matrix/client`: for Client-Server discovery
- `/.well-known/matrix/server`: for Server-Server (federation) discovery
- `/.well-known/matrix/support`: admin contact details (strongly recommended to have)
- `/.well-known/matrix/support`: admin contact details (strongly recommended to have). By default, this include members of the admin room.
Member

"to have" can be dropped while we're at it, and for formatting, I would place "By default, [...]" on its own line with no terminating or separating periods

"to have" can be dropped while we're at it, and for formatting, I would place "By default, [...]" on its own line with no terminating or separating periods
stratself marked this conversation as resolved
@ -129,2 +133,2 @@
:::warning
Even if you choose to serve the well-known files manually, if you are using delegation at all, you **must** still set the `global.well_known.client` configuration option to the domain you're delegating to. Continuwuity needs to know the domain it runs on for OAuth-compatible clients to work correctly.
:::warning Warning: always configure [global.well_known]
Even if you choose to serve the well-known files manually, you **must** still set the `global.well_known.client` configuration option to the domain you're delegating to. Continuwuity needs to know the domain it runs on, in order to redirect clients properly to email, OAuth, and account management endpoints.
Member

There should be no separating comma: "runs on in order to"

There should be no separating comma: "runs on in order to"
stratself marked this conversation as resolved
@ -196,0 +200,4 @@
- [Federation Setup guide by MTRNord](https://connectivity-tester.mtrnord.blog/docs/getting-started/federation-setup)
See the following Matrix Specs for full details on client/server resolution mechanisms:
Member

"sections of the Matrix Specification for more details on client and server resolution mechanisms:"

"sections of the Matrix Specification for more details on client and server resolution mechanisms:"
stratself marked this conversation as resolved
@ -196,0 +202,4 @@
See the following Matrix Specs for full details on client/server resolution mechanisms:
- [Server-to-Server resolution](https://spec.matrix.org/latest/server-server-api/#resolving-server-names) (see this for more information on SRV records)
Member

The ending of this can just be ", particularly for more information on SRV records"

The ending of this can just be ", particularly for more information on SRV records"
Author
Member

Removed entirely as SRV records are in Appendix

Removed entirely as SRV records are in Appendix
stratself marked this conversation as resolved
@ -196,0 +210,4 @@
### Docker Compose examples
The following Compose files are taken from [Docker instructions](../deploying/docker.mdx) and reconfigured to support split-domain delegation. Note the updated `CONTINUWUITY_WELL_KNOWN` variable and relevant changes in reverse proxy rules.
Member

"from the [Docker deployment instructions]"

"from the [Docker deployment instructions]"
stratself marked this conversation as resolved
@ -196,0 +220,4 @@
```
```txt title="Caddyfile"
https://matrix.example.com:443 {
Member

The same feedback I have given everywhere else about not needing to specify https:// or :443 is relevant here.

The same feedback I have given everywhere else about not needing to specify https:// or :443 is relevant here.
stratself marked this conversation as resolved
@ -197,3 +262,2 @@
:::warning
The following methods are **not recommended** due to increased complexity with little benefits. If you have already set up `.well-known` delegation as above, you can safely skip this part.
:::
The following methods use DNS SRV records, and are **not recommended** due to increased complexity with little benefits. They also work with federation traffic only.
Member

"due to their increased complexity with few benefits" and "Additionally, they only work for federation traffic"

"due to their increased complexity with few benefits" and "Additionally, they only work for federation traffic"
stratself marked this conversation as resolved
@ -200,2 +263,3 @@
The following methods use DNS SRV records, and are **not recommended** due to increased complexity with little benefits. They also work with federation traffic only.
The following methods uses SRV DNS records and only work with federation traffic. They are only included for completeness.
If you have already set up `.well-known` delegation as above, you can safely skip this part.
Member

".well-known delegation" and "well-known delegation" should be consistent across the docs

"`.well-known` delegation" and "well-known delegation" should be consistent across the docs
stratself marked this conversation as resolved
@ -19,3 +19,3 @@
### For Docker users
Docker bridge networks uses a non-performant resolver to intercept and respond to container hostnames, and **this should also be avoided**. Instead, mount a custom `/etc/resolv.conf` file into the container, and hardcode a resolver address to bypass Docker's.
By default, Docker bridge networks uses a non-performant resolver to intercept and respond to container hostnames. **This resolver should also be avoided**. Instead, mount a custom `/etc/resolv.conf` file into the container, and hardcode a resolver address to bypass Docker's.
Member

"non-performant" should be replaced with "poorly performing" while we're here

"non-performant" should be replaced with "poorly performing" while we're here
stratself marked this conversation as resolved
@ -29,2 +29,3 @@
networks:
matrix_net:
# re-use the `proxy` network, as shown in Docker deployment docs,
Member

"in the [...]"

"in the [...]"
stratself marked this conversation as resolved
@ -54,1 +60,4 @@
:::warning Breaking resolution with container hostnames
By mounting a custom `/etc/resolv.conf`, you won't be able to connect to other container via their hostnames, such as [appservice](../appservices.mdx) in the same Docker network. To work around this, also force your appservice to use a static IP address, and refer to such IP during registration.
Member
After you mount a custom `/etc/resolv.conf`, you won't be able to connect to other container using their hostnames, such as for [appservices](../appservices.mdx) in the same Docker network. To work around this, force your appservice to use a static IP address, and refer to its static IP during registration.
``` After you mount a custom `/etc/resolv.conf`, you won't be able to connect to other container using their hostnames, such as for [appservices](../appservices.mdx) in the same Docker network. To work around this, force your appservice to use a static IP address, and refer to its static IP during registration. ```
stratself marked this conversation as resolved
@ -55,3 +93,3 @@
### For IPv4-only users
If you don't have IPv6 connectivity, changing `ip_lookup_strategy` to only resolve for IPv4 will reduce unnecessary AAAA queries.
If you don't have IPv6 connectivity, change `ip_lookup_strategy` to `1` in Continuwuity's configuration, in order to only resolve for IPv4 and remove unnecessary AAAA queries.
Member

Again, there need not be a separating comma: "configuration in order to exclusively resolve IPv4 addresses and remove unnecessary AAAA queries."

Again, there need not be a separating comma: "configuration in order to exclusively resolve IPv4 addresses and remove unnecessary AAAA queries."
Member

i would additionally reword it to

in order to only resolve for IPv4 addresses and remove unnecessary AAAA IPv6 queries.

i would additionally reword it to > in order to only resolve ~~for~~ IPv4 addresses and remove unnecessary ~~AAAA~~ IPv6 queries.
stratself marked this conversation as resolved
@ -78,2 +115,3 @@
### Using a forwarder (optional)
Unbound by default employs **recursive resolution** and contacts many servers around the world. While this allows updated and authoritative answers and are generally viable for most users, sometimes these recursive queries can be too slow to fully resolve. As an alternative, you can consider **forwarding** your queries to public resolvers, and benefit from faster responses from their CDNs.
Unbound by default employs **recursive resolution** and contacts many nameservers around the world. While this is generally viable for most users, sometimes these recursive queries can be too slow to fully resolve. As an alternative, you may consider **forwarding** your queries to public resolvers, and benefit from faster responses from their CDNs.
Member
Unbound employs **recursive resolution** by default, contacting many nameservers around the world in the process. While this is generally viable for most users, sometimes these recursive queries can be too slow to fully resolve. As an alternative, you may consider **forwarding** your queries to public resolvers, and benefit from faster responses from their caches.
``` Unbound employs **recursive resolution** by default, contacting many nameservers around the world in the process. While this is generally viable for most users, sometimes these recursive queries can be too slow to fully resolve. As an alternative, you may consider **forwarding** your queries to public resolvers, and benefit from faster responses from their caches. ```
stratself marked this conversation as resolved
@ -145,3 +183,3 @@
## Testing
As a rough stress test, you can run `!admin query resolver flush-cache -a` or `!admin server clear-caches` to trigger a netburst of DNS queries. If your resolver can handle these loads without problem, then it should be ready for regular Continuwuity activity.
As a rough stress test, you can run `!admin query resolver flush-cache -a` or `!admin server clear-caches` to clear your destinations cache - this will then trigger a netburst of DNS queries. If your resolver can handle this load without problem, then it should be ready for regular Continuwuity activity.
Member

While we're here, "without problem" should be "without issue" or "without a problem". This is a common malformation.

While we're here, "without problem" should be "without issue" or "without a problem". This is a common malformation.
stratself marked this conversation as resolved
@ -7,3 +7,3 @@
## DNS tuning (recommended)
Please see the dedicated [DNS tuning guide](./dns.mdx).
For federating servers, DNS tuning is important for a smooth experience. Please see the dedicated [DNS tuning guide](./dns.mdx).
Member

"DNS tuning is important to ensure a smooth experience for federated servers."

"DNS tuning is important to ensure a smooth experience for federated servers."
stratself marked this conversation as resolved
@ -46,4 +47,4 @@
### Compression
Some RocksDB settings can be adjusted such as the compression method chosen. See
Member

There should be a separating comma for "adjusted, such as" while we're here

There should be a separating comma for "adjusted, such as" while we're here
stratself marked this conversation as resolved
@ -43,3 +43,3 @@
via liburing for improved read performance.
RocksDB troubleshooting can be found [in the RocksDB section of troubleshooting](troubleshooting.md).
RocksDB troubleshooting can be found [in the RocksDB section of
Member

Once again, inconsistent wrapping.

Once again, inconsistent wrapping.
Author
Member

How would you like it to wrap? It works though

How would you like it to wrap? It works though
stratself marked this conversation as resolved
@ -50,1 +51,3 @@
the RocksDB section in the [example config](./reference/config.mdx).
the RocksDB section in the [example config](./reference/config.mdx), and the
[database compression section](./guides/performance.mdx#tuning-database-compression)
in the perf-tuning documentation for more.
Member

"in the performance tuning documentation"

"in the performance tuning documentation"
stratself marked this conversation as resolved
@ -85,1 +88,3 @@
same. This is unfortunately a bad design choice by Facebook as we are using the
### Database online backup
If you'd like to backup your database online without any downtime, see the
Member
If you'd like to run an online backup of your database \- that is, a backup with no downtime \- check the [`!admin server` command](./reference/admin/server.md) for the backup commands and the `database_backup_path` config options in the example config.
``` If you'd like to run an online backup of your database \- that is, a backup with no downtime \- check the [`!admin server` command](./reference/admin/server.md) for the backup commands and the `database_backup_path` config options in the example config. ```
stratself marked this conversation as resolved
@ -86,0 +91,4 @@
[`!admin server` command](./reference/admin/server.md) for the backup commands
and the `database_backup_path` config options in the example config.
Please note that the format of the database backup is not the exact same.
Member
Please note that the format of online backups is not the same as the format of offline backups. [...]
``` Please note that the format of online backups is not the same as the format of offline backups. [...] ```
stratself marked this conversation as resolved
@ -132,2 +143,2 @@
immutable for all media requests (download and thumbnail) to reduce unnecessary
media requests from browsers, reduce bandwidth usage, and reduce load.
While Continuwuity does not implement built-in S3 support, using an "S3 filesystem
on the `media/` path will works. Continuwuity also sends a `Cache-Control` header
Member

"will work"

"will work"
stratself marked this conversation as resolved
@ -133,1 +143,3 @@
media requests from browsers, reduce bandwidth usage, and reduce load.
While Continuwuity does not implement built-in S3 support, using an "S3 filesystem
on the `media/` path will works. Continuwuity also sends a `Cache-Control` header
of 1 year and immutable for all media requests (download and thumbnail)
Member

"and immutable for all media requests" needs to be rephrased

"and immutable for all media requests" needs to be rephrased
Author
Member

rephrased

rephrased
stratself marked this conversation as resolved
@ -1,3 +1,8 @@
## Configuration file reference
This configuration file reflect latest developments on the `main` branch. To check the file from a specific Continuwuity version, browse the [tags](https://forgejo.ellis.link/continuwuation/continuwuity/tags) and view the `conduwuit-example.toml` file there.
Member
This configuration file reflects the latest developments on the `main` branch. To check the configuration for a specific Continuwuity version, browse the [tags](https://forgejo.ellis.link/continuwuation/continuwuity/tags) and navigate to the `conduwuit-example.toml` file for that version.
``` This configuration file reflects the latest developments on the `main` branch. To check the configuration for a specific Continuwuity version, browse the [tags](https://forgejo.ellis.link/continuwuation/continuwuity/tags) and navigate to the `conduwuit-example.toml` file for that version. ```
Member

I'd argue the ending should be "for that version"

I'd argue the ending should be "for that version"
Member

Updated to reflect silver's feedback.

Updated to reflect silver's feedback.
stratself marked this conversation as resolved
@ -61,1 +61,3 @@
## Reverse proxying well-known files to Continuwuity
## Serving well-known files
You can either reverse proxy well-known paths to Continuwuity, or serve static JSON files yourself.
Member

Would add a "the", to become "reverse proxy the well-known paths"

Would add a "the", to become "reverse proxy the well-known paths"
stratself marked this conversation as resolved
@ -30,1 +30,3 @@
matrix_net:
# re-use the `proxy` network, as shown in Docker deployment docs,
# with a custom subnet
Member

do these four words specifically need to reside on a different line? it looks kinda funky

image

do these four words specifically need to reside on a different line? it looks kinda funky ![image](/attachments/6d0eb1c3-d2d4-4031-a259-5b3e25f92360)
stratself marked this conversation as resolved
@ -28,3 +28,3 @@
```yaml title="docker-compose.yml"
networks:
matrix_net:
Member

stray newline

image

stray newline ![image](/attachments/bd03126a-dd30-4bdb-85aa-c2d27cd82063)
stratself marked this conversation as resolved
s1lv3r approved these changes 2026-08-04 20:40:22 +00:00
s1lv3r left a comment

Approved minus the few comments made

Approved minus the few comments made
stratself force-pushed stratself/docs-sweepfixes-ii from 88c1d6cc8d
All checks were successful
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Check changed files (pull_request) Successful in 5s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 55s
to 1fadd67e54
All checks were successful
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 4s
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 56s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Update flake hashes / update-flake-hashes (pull_request) Successful in 1m8s
2026-08-05 05:11:24 +00:00
Compare
Author
Member

Changes made except one outstanding. I took the liberty to do s/docker-compose/docker compose for the livekit docs as well

Changes made ~~except one outstanding~~. I took the liberty to do `s/docker-compose/docker compose` for the livekit docs as well
seercat requested changes 2026-08-05 05:50:18 +00:00
Dismissed
@ -194,2 +200,3 @@
## Related Documentation
## Using SRV records (not recommended)
Community guides:
First-time contributor

### Community guides, move after the matrix spec stuff (next comment)

`### Community guides`, move after the matrix spec stuff (next comment)
stratself marked this conversation as resolved
@ -196,0 +203,4 @@
- [Federation Setup guide by MTRNord](https://connectivity-tester.mtrnord.blog/docs/getting-started/federation-setup)
See the following sections of the Matrix Specification for more details on client and server resolution mechanisms:
First-time contributor

Replace line with ### Matrix Specification, move above community guides

Replace line with `### Matrix Specification`, move above community guides
Author
Member

Sectioned as recommended

Sectioned as recommended
stratself marked this conversation as resolved
fix(docs): Incorporate change requests from PR comments II
All checks were successful
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 6s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 6s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m12s
43ed314b93
seercat approved these changes 2026-08-05 10:26:13 +00:00
@ -54,1 +59,4 @@
:::warning Breaking resolution with container hostnames
After you mount a custom `/etc/resolv.conf`, you won't be able to connect to other container using their hostnames, such as for [appservices](../appservices.mdx) in the same Docker network. To work around this, force your appservice to use a static IP address, and refer to its static IP during registration.
Member

"other containers"

"other containers"
stratself marked this conversation as resolved
fix: Small grammar
All checks were successful
Documentation / Build and Deploy Documentation (pull_request) Has been skipped
Checks / Prek / Check changed files (pull_request) Successful in 6s
Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s
Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped
Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m11s
1fc8bd1f9f
Sign in to join this conversation.
No milestone
No project
No assignees
5 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
continuwuation/continuwuity!2083
No description provided.