Skip to content

docs(self-hosting): document COMPOSE_PROFILES for the bundled postgres - #154

Merged
charlesrhoward merged 1 commit into
mainfrom
docs/compose-external-db
Oct 15, 2026
Merged

charlesrhoward merged 1 commit into
mainfrom
docs/compose-external-db

Conversation

@charlesrhoward

@charlesrhoward charlesrhoward commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Documents Mogplex/mogplex#652: the bundled Postgres runs under the postgres compose profile (COMPOSE_PROFILES=postgres in .env.example). External-database users remove it, and then no local Postgres starts.

  • Environment variables: adds COMPOSE_PROFILES to the Docker Compose table.
  • Quickstart: the postgres service row explains the switch.

Merge order: after Mogplex/mogplex#652 merges.

pnpm types:check passes.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@vercel

vercel Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
mogplex-docs Ready Ready Preview Oct 15, 2026 10:36pm UTC

Request Review

@mogplex

mogplex Bot commented Oct 15, 2026

Copy link
Copy Markdown

Mogplex PR Review

Status: No material issues found

Docs-only change that accurately documents the postgres compose profile for the bundled Postgres in both the environment-variables table and the quickstart service table. No factual, security, or correctness problems found; approve-ready once the referenced Mogplex/mogplex#652 merges.

  • content/docs/self-hosting/environment-variables.mdx:L163 — Adds COMPOSE_PROFILES to the Docker Compose table with the correct guidance: postgres starts the bundled Postgres, and it must be removed when DATABASE_URL points at an external database.
  • content/docs/self-hosting/quickstart.mdx:L90 — The postgres service row accurately explains the profile switch and the external-database path ("set DATABASE_URL and DATABASE_URL_UNPOOLED and remove COMPOSE_PROFILES").
  • Both changes stay consistent with the existing table style and wording of each page, and they match the behavior described in fix(self-host): external DATABASE_URL no longer starts bundled postgres mogplex#652.
  • The PR body correctly notes the merge-order dependency on fix(self-host): external DATABASE_URL no longer starts bundled postgres mogplex#652; until that lands, these docs would describe behavior that doesn't exist yet, so holding this PR is the right call.

View check run

@charlesrhoward
charlesrhoward merged commit 8973fba into main Oct 15, 2026
7 checks passed
@charlesrhoward
charlesrhoward deleted the docs/compose-external-db branch October 15, 2026 22:39

This branch was successfully deployed

1 active deployment
Preview — 42e1c223 Deployed Oct 15, 2026 by vercel[bot]
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.

1 participant