feat-004: Account Registration & Onboarding#1
Closed
leecampbell-codeagent wants to merge 4 commits intomainfrom
Closed
feat-004: Account Registration & Onboarding#1leecampbell-codeagent wants to merge 4 commits intomainfrom
leecampbell-codeagent wants to merge 4 commits intomainfrom
Conversation
…at-004) - Multi-step onboarding wizard (welcome, role selection, profile, preferences, completion) - Account domain entity extended with onboarding_step, role, profile, and notification preference fields - Domain event emission to event store on onboarding completion and profile updates - API endpoints: GET /me, PATCH /me/onboarding, PATCH /me, PATCH /me/preferences - PgAccountRepository updated with findById and update methods - InMemoryEventStore and PgEventStoreAdapter implemented - Frontend: OnboardingWizard with StepProgress, RoleCards, ToggleSwitch components - Settings pages: /settings/profile and /settings/preferences - Onboarding guard redirects users to /onboarding if not complete - 260 tests passing (132 backend + 128 frontend) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add 401 unauthenticated tests for all 4 account API endpoints - Fix preferences_updated event payload to include categories_changed array - Wrap account update + event append in shared database transaction (Edge Case LeeCampbell#17) - Introduce TransactionPort and PgTransactionAdapter for transactional DB operations Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Exploratory review: PASS (code-review based, all 8 US verified) - Security review: PASS (0 critical/high findings) - Audit: PASS (all 3 failures resolved in iteration 2) - CI/CD: PASS — add GitHub Actions workflow (lint, typecheck, test, build, audit) - Fix 34 lint/a11y errors: semantic HTML in onboarding components (label+input for radio group, section for region, output for status), biome format fixes Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
9bc3091 to
774b35a
Compare
This was referenced Mar 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
onboarding_step,role, profile fields (display_name,bio,avatar_url), and notification preferences; all mutations are immutable and emit domain events to the event storeGET /api/v1/accounts/me,PATCH /api/v1/accounts/me/onboarding,PATCH /api/v1/accounts/me,PATCH /api/v1/accounts/me/preferencesOnboardingWizardwithStepProgress,RoleCard,ToggleSwitchcomponents; settings pages at/settings/profileand/settings/preferences; onboarding guard redirects users to/onboardinguntil completeQuality Gate
Reports
.claude/reports/feat-004-exploratory.md.claude/reports/feat-004-security.md.claude/reports/feat-004-audit.md.claude/reports/feat-004-cicd.md🤖 Generated with Claude Code