Add general settings page #168

New issue

Closed

opened 2026-06-20 08:48:13 -05:00 by erik · 3 comments

erik commented

2026-06-20 08:48:13 -05:00

Owner

Goal

Create a general settings page for user profile and appearance settings.

Requirements

Replace the redundant overview settings page with a general settings page.
Allow the user to update their Display Name.
Display the signed-in user's email address as read-only login identity.
Move the dark/light mode toggle to the general settings page.
Remove the dark/light mode toggle from the main header menu.

Acceptance criteria

A general settings page exists in place of the overview settings page.
Users can change their Display Name from the general settings page.
The signed-in user's email address is displayed read-only on the general settings page.
The dark/light mode toggle appears on the general settings page.
The dark/light mode toggle no longer appears in the main header menu.

Dependencies

None

## Goal Create a general settings page for user profile and appearance settings. ## Requirements - Replace the redundant overview settings page with a general settings page. - Allow the user to update their Display Name. - Display the signed-in user's email address as read-only login identity. - Move the dark/light mode toggle to the general settings page. - Remove the dark/light mode toggle from the main header menu. ## Acceptance criteria - [ ] A general settings page exists in place of the overview settings page. - [ ] Users can change their Display Name from the general settings page. - [ ] The signed-in user's email address is displayed read-only on the general settings page. - [ ] The dark/light mode toggle appears on the general settings page. - [ ] The dark/light mode toggle no longer appears in the main header menu. ## Dependencies - None

erik added the

status:inprogress

priority:medium

labels

2026-06-20 08:48:13 -05:00

erik commented

2026-06-20 08:48:13 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T13:45:17.286Z

Update

Started implementation after approved plan. Scope: replace settings overview with General settings, let signed-in users update display name and email, move color mode control to General settings, remove header color-mode toggle, and update tests.

_Synced from todu comment by @todu on 2026-06-20T13:45:17.286Z_ ### Update Started implementation after approved plan. Scope: replace settings overview with General settings, let signed-in users update display name and email, move color mode control to General settings, remove header color-mode toggle, and update tests.

erik commented

2026-06-20 08:53:44 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T13:53:27.515Z

Review update

PR: #169 is open and reviewed.
Scope was adjusted per request: email is displayed read-only; only Display Name is editable.
Result: approved; no blocking issues or warnings.
Verified: npm run build --workspace @slugkit/template-site, targeted template-site app/auth/public/passkeys/site-user tests, ./scripts/pre-pr.sh, and Forgejo Actions run #235.
Acceptance: 5/5 criteria met.
Follow-up: waiting for explicit human merge approval.

_Synced from todu comment by @todu on 2026-06-20T13:53:27.515Z_ ### Review update - PR: https://forge.caradoc.com/erik/slugkit/pulls/169 is open and reviewed. - Scope was adjusted per request: email is displayed read-only; only Display Name is editable. - Result: approved; no blocking issues or warnings. - Verified: `npm run build --workspace @slugkit/template-site`, targeted template-site app/auth/public/passkeys/site-user tests, `./scripts/pre-pr.sh`, and Forgejo Actions run #235. - Acceptance: 5/5 criteria met. - Follow-up: waiting for explicit human merge approval.

erik

2026-06-20 08:59:12 -05:00

closed this issue
added
status:done
and removed
status:inprogress
labels

erik commented

2026-06-20 08:59:12 -05:00

Author

Owner

Synced from todu comment by @todu on 2026-06-20T13:55:57.097Z

Closing summary

PR #169 was merged into main.

Acceptance criteria:

General settings page exists in place of the overview settings page — met via /settings rendering General profile/appearance settings.
Users can change their Display Name — met via POST /settings and updateSiteUserDisplayName.
Signed-in email address is displayed read-only — met; email remains login identity and is not editable.
Dark/light mode toggle appears on General settings — met in the Appearance section.
Dark/light mode toggle no longer appears in the main header menu — met for public/settings headers while color-mode script support remains.

Verification:

npm run build --workspace @slugkit/template-site
Targeted template-site app/auth/public/passkeys/site-user tests
./scripts/pre-pr.sh
Forgejo Actions run #235 passed.

Readiness: READY. Closing task.

_Synced from todu comment by @todu on 2026-06-20T13:55:57.097Z_ ### Closing summary PR #169 was merged into `main`. Acceptance criteria: - General settings page exists in place of the overview settings page — met via `/settings` rendering General profile/appearance settings. - Users can change their Display Name — met via `POST /settings` and `updateSiteUserDisplayName`. - Signed-in email address is displayed read-only — met; email remains login identity and is not editable. - Dark/light mode toggle appears on General settings — met in the Appearance section. - Dark/light mode toggle no longer appears in the main header menu — met for public/settings headers while color-mode script support remains. Verification: - `npm run build --workspace @slugkit/template-site` - Targeted template-site app/auth/public/passkeys/site-user tests - `./scripts/pre-pr.sh` - Forgejo Actions run #235 passed. Readiness: READY. Closing task.