Skip to content

Conversation

@initstring
Copy link
Owner

Motivation

  • Make the Docker installation flow more logical by forcing configuration of the env file before starting containers.
  • Shorten and clarify authentication guidance so it's clear how to choose between Google SSO and demo mode.
  • Warn explicitly that demo mode allows anyone who can reach the sign-in page to log in without an account.
  • Standardize environment variable names used by Docker to remove legacy RTAP_* prefixes for clarity.

Description

  • Rewrote docs/installation.md to configure the .env first, then choose authentication mode, and finally start Docker, and tightened the auth section with a clear demo-mode warning.
  • Updated deploy/docker/.env.example to use unified variable names (AUTH_URL, AUTH_SECRET, INITIAL_ADMIN_EMAIL, ENABLE_DEMO_MODE, POSTGRES_*, PORT, DATABASE_URL, etc.) and adjusted defaults/comments accordingly.
  • Updated deploy/docker/docker-compose.yml to reference the new environment variable names and to set a pre-wired DATABASE_URL that points at the postgres container.
  • Updated docs/development.md to add repository-root context for commands and an Environment variables section noting the shared env names and TEST_DATABASE_URL for tests.

Testing

  • No automated tests were run for these changes.
  • The changes are documentation and configuration-only and were validated by local editing and commit.

Codex Task

@initstring
Copy link
Owner Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@initstring initstring merged commit 80c3260 into main Jan 17, 2026
5 checks passed
@initstring initstring deleted the codex/standardize-environment-variable-naming branch January 17, 2026 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants