Corentin
8e717401bd
Per Corentin's feedback (2026-05-08): .env should be reserved for server-side config (SMTP, DB, OIDC). Branding (name, colors, logo) is admin/UI territory. Changes: - Remove BRAND_NAME / BRAND_LOGO_URL / BRAND_PRIMARY_COLOR / BRAND_ACCENT_COLOR from .env.example, vite.config.ts define block - Hardcode "AcadeDoc" + #2563eb / #7c3aed as defaults in apps/client/src/lib/config.ts and brand-theme.ts - getBrandTheme() now takes optional runtime overrides instead of reading process.env (used by per-workspace branding hook to apply DB-stored colors) - Server getMailFromName() defaults to "AcadeDoc" hardcoded; only MAIL_FROM_NAME env var can override - Fix workspace-branding.spec.ts (was importing vitest in jest project, R4.4 leftover bug, similar to Patch 017 scope) - Fix environment.service.spec.ts (was missing ConfigService provider in TestingModule, pre-existing upstream bug surfaced by jest run) Tests: 13 brand-theme + 13 workspace-branding + 2 environment = 28 green. Per-workspace UI override via /settings/branding (R4.4) works unchanged. Patch 020. |
||
|---|---|---|
| .. | ||
| audit | ||
| environment | ||
| export | ||
| health | ||
| import | ||
| queue | ||
| redis | ||
| security | ||
| static | ||
| storage | ||
| telemetry | ||
| throttle | ||
| transactional | ||