Mark/docker_backup

Fork 0

Add authentication and user management — Phase 4: Files Affected #41

New issue

Closed

opened 2026-05-28 00:47:24 +00:00 by Mark · 1 comment

Mark commented

2026-05-28 00:47:24 +00:00

Owner

Phase 4: Files Affected

src/pages/Settings.tsx (add user management UI)
src/store/index.ts (auth state)
src/lib/api.ts (auth endpoints)
New: src/store/auth.ts

Decomposed from docker_backup#3

## Phase 4: Files Affected - `src/pages/Settings.tsx` (add user management UI) - `src/store/index.ts` (auth state) - `src/lib/api.ts` (auth endpoints) - New: `src/store/auth.ts` --- _Decomposed from [docker_backup#3](https://forgejo.tail018ac4.ts.net/Mark/docker_backup/issues/3)_

Mark added the

needs-triage

label

2026-05-28 00:47:24 +00:00

Mark referenced this issue

2026-05-28 00:47:27 +00:00

Add authentication and user management #3

Mark added

blocked:needs-decision

and removed

needs-triage

labels

2026-05-28 00:50:53 +00:00

Mark commented

2026-05-28 00:50:53 +00:00

Author

Owner

⏸ Needs Decision

Blocking factors:

Didn't meet auto-approve criteria — scope=small, risk=medium, spec=aligned

Scope: small Risk: medium Spec: aligned Type: feature
Summary: Add user management UI to the Settings page, auth state management to the Zustand store (new module + store/index.ts), and auth-related API endpoints.
Suggestion: Implement a separate Zustand auth slice in src/store/auth.ts (following the planned new file), expose auth mutations and queries via src/lib/api.ts, and add a 'User Management' section to the existing Settings page (src/pages/Settings.tsx) using shadcn/ui table/form components. Use React 19's useActionState for form actions. Run npm run build after implementation to verify TS compiles cleanly.
Rationale: Small scope (3 existing files + 1 new file, no new subsystem). Medium risk because authentication is security-sensitive and the project has no test coverage per AGENTS.md. Aligned with project patterns (Zustand state, shadcn/ui, React 19 patterns) and the decomposed Phase 4 plan from the original issue #3.

Action: Review each blocking factor above. Remove blocked:needs-decision label to route back to triage.

## ⏸ Needs Decision **Blocking factors:** - **Didn't meet auto-approve criteria** — scope=small, risk=medium, spec=aligned --- **Scope:** small **Risk:** medium **Spec:** aligned **Type:** feature **Summary:** Add user management UI to the Settings page, auth state management to the Zustand store (new module + store/index.ts), and auth-related API endpoints. **Suggestion:** Implement a separate Zustand auth slice in src/store/auth.ts (following the planned new file), expose auth mutations and queries via src/lib/api.ts, and add a 'User Management' section to the existing Settings page (src/pages/Settings.tsx) using shadcn/ui table/form components. Use React 19's useActionState for form actions. Run npm run build after implementation to verify TS compiles cleanly. **Rationale:** Small scope (3 existing files + 1 new file, no new subsystem). Medium risk because authentication is security-sensitive and the project has no test coverage per AGENTS.md. Aligned with project patterns (Zustand state, shadcn/ui, React 19 patterns) and the decomposed Phase 4 plan from the original issue #3. **Action:** Review each blocking factor above. Remove `blocked:needs-decision` label to route back to triage.

Mark

2026-05-28 07:18:39 +00:00

closed this issue
added
triage-approved
and removed
blocked:needs-decision
labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Mark/docker_backup#41

No description provided.

Rows
Columns