Skip to content

feat(dashboard): replace raw .env editor with Configuration & Deployment Guide card - #341

Merged
trefeon merged 1 commit into
mainfrom
feat/settings-guide-card
Sep 4, 2026
Merged

feat(dashboard): replace raw .env editor with Configuration & Deployment Guide card#341
trefeon merged 1 commit into
mainfrom
feat/settings-guide-card

Conversation

@trefeon

@trefeon trefeon commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

Replaces the raw .env textarea editor with an informative Configuration File & Deployment Guide card (Option B):

  • Removes RawEditor Textarea:
    • Direct .env text editing in web browsers is an anti-pattern that caused operator confusion and risk of malformed configurations.
  • Adds ConfigFileGuideCard (SettingsCard + SettingsRow template):
    • File Location: Displays /app/state/.env with copy button and explains container mounting.
    • Precedence Order: Visual flow showing Host Env > -config JSON > .env > Defaults.
    • Host Terminal Commands: Copyable nano .env and docker compose up -d with instructions for host-level edits.
  • Verification:
    • Updated e2e test in ux.spec.ts asserting guide card rendering and host command copy buttons.
    • Passed all 5 settings tests and full hermetic backend suite.
    • Rebuilt embedded frontend dist bundle.

…ent Guide card

- Remove textarea raw .env editor (RawEditor.svelte) from Settings.
- Add ConfigFileGuideCard (Option B) explaining file location (/app/state/.env),
  precedence order (Host Env > -config JSON > .env > Defaults), and host terminal
  commands with copyable buttons (nano .env, docker compose up -d).
- Update ux.spec.ts e2e test to verify the guide card and host commands.
- Rebuild embedded dist bundle; all tests pass.
Copilot AI balanced review requested due to automatic review settings September 4, 2026 11:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@trefeon
trefeon merged commit 7888d33 into main Sep 4, 2026
4 of 5 checks passed
@trefeon
trefeon deleted the feat/settings-guide-card branch September 4, 2026 11:58
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.

2 participants