Block a user
feat(planner): wire variety-aware suggestions into RecipePicker for empty slots
feat(planner): wire variety-aware suggestions into RecipePicker for empty slots
feat(variety): implement C3 warning cards with recipe names and swap links (V1)
feat(settings): implement E4 variety score settings (J9)
feat(settings): implement /settings page — Kachel-Ansicht (E1)
🎨 Atlas — UI/UX Design
Questions & Observations
2fr 1frGrid — Proportionen in der Praxis. Bei einer typischen Desktop-Breite von ~900px Content-Bereich ergibt2fr 1frca.…
feat(settings): implement /settings page — Kachel-Ansicht (E1)
🧪 QA Engineer
Questions & Observations
Fehlende Akzeptanzkriterien:
- Was passiert wenn
GET /v1/ingredient-categoriesfehlschlägt? Die Issue-Beschreibung sagt?? 0als…
feat(settings): implement /settings page — Kachel-Ansicht (E1)
🔒 Sable — Security Engineer
Questions & Observations
- Zugriffskontrolle auf
/settings. Die Spec erwähnt keine Rollen-Einschränkung für E1. Sollte/settingsfürmitglied…
feat(settings): implement /settings page — Kachel-Ansicht (E1)
🏗️ Backend Engineer — Spring Boot / PostgreSQL
Questions & Observations
- Keine neuen Backend-Endpoints nötig — aber einen Blick wert. E1 selbst braucht nur `GET /v1/ingredient-cate…
feat(settings): implement /settings page — Kachel-Ansicht (E1)
⚛️ Kai — Frontend Engineer
Questions & Observations
- memberCount: woher? Das Issue sagt "aus layout data oder separatem Call".
+layout.server.tsgibt aktuellbenutzerund…
feat(planner): wire variety-aware suggestions into RecipePicker for empty slots
Review concerns addressed
All blockers and suggestions from the multi-persona review have been resolved. Here's a summary by concern:
✅ SwapSuggestionList deleted — RecipePicker…
marcel
pushed to feat/issue-46-wire-suggestions-recipe-picker at marcel/mealprep
2026-04-09 15:10:05 +02:00
feat(members): implement /members page — Kachel-Ansicht (E2)
🎨 Atlas — UI/UX Design
Questions & Observations
- Typografie-Konsistenz. Button-Text im Dropdown und im Dialog muss exakt sein:
font-size: 13px,font-weight: 500, `letter-spacing…
feat(members): implement /members page — Kachel-Ansicht (E2)
🧪 QA Engineer
Questions & Observations
Fehlende Akzeptanzkriterien für wichtige Edge Cases:
- **Was passiert, wenn der Planer versucht sich selbst zu degradieren, aber noch ein…
feat(members): implement /members page — Kachel-Ansicht (E2)
🔒 Sable — Security Engineer
Questions & Observations
- IDOR auf DELETE und PATCH — kritisch.
DELETE /v1/households/mine/members/{userId}muss serverseitig verifizieren, dass…
feat(members): implement /members page — Kachel-Ansicht (E2)
🏗️ Backend Engineer — Spring Boot / PostgreSQL
Questions & Observations
- Drei neue Endpoints — wo sitzen die Business Rules? Die Guards "Planer kann sich nicht selbst entfernen"…
feat(members): implement /members page — Kachel-Ansicht (E2)
⚛️ Kai — Frontend Engineer
Questions & Observations
- Backend-first dependency is a hard blocker. DELETE + PATCH member + GET invites müssen alle drei fertig und im
schema.d.ts…
feat(settings): implement /settings page — Kachel-Ansicht (E1)
feat(members): implement /members page — Kachel-Ansicht (E2)