feat/issue-751-journey-item-crud-api #791
Reference in New Issue
Block a user
Delete Branch "feat/issue-751-journey-item-crud-api"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
DocumentSummary: lean document projection for journey item embedding — skips tag-color resolution (getSummaryById), includes receiverCount (0 when no receivers, non-null). JourneyItemView: response record for item CRUD and GET. GeschichteView: detail response with summarised author {id, displayName} to prevent AppUser email/group leak. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>doesNotExist() asserts the key is absent from the JSON object, but Jackson serializes a null Optional<String> as {"note": null} — the key is present with a null value. nullValue() correctly matches that case. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>The action was writing aria-checked directly and then firing onChange, which also triggered Svelte's own aria-checked={selected === type} binding. Double-ownership: action now only calls focus() + onChange(value); Svelte owns the attribute update. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>geschichten components now describe the type-based reader split (StoryReader / JourneyReader / JourneyItemCard / JourneyInterlude), the TypeSelector creation flow, and the full set of API endpoints (including DELETE /api/geschichten/{id} and GET /api/persons/{id} for person pre-population). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>Importing layout.css in test-setup.ts activated Tailwind's responsive breakpoint classes (hidden lg:flex, hidden md:block, etc.), making 42 elements invisible at the default narrow Playwright test viewport. Revert the CSS import. Instead, add inline style attributes to the three components whose tests measure computed properties (min-height, font-size) — these values match what the Tailwind classes produce, so the real app appearance is unchanged. Also fix goto mock leakage in the geschichten/[id] delete-failure test: the delete-success test's goto('/geschichten') call was not cleared before the failure test ran. Add beforeEach(vi.clearAllMocks) to reset mock state. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>