WompSchmiede/FamilienFeierPlaner
SHA256
1
1
Fork 0
Code Issues 18 Pull requests Projects Releases 5 Packages 3 Wiki Activity Actions

"Antworten" im Gästeliste-Dropdown öffnet GuestFormModal + Refactoring #158

New issue
Closed
opened 2026-06-30 23:50:52 +02:00 by niboer · 2 comments
niboer commented 2026-06-30 23:50:52 +02:00
Owner
Copy link

Problem

Im Gästeliste-Dropdown (?tab=guests) gibt es das Submenü "Status". Der Eintrag "Antworten" setzt aktuell direkt status=confirmed via ?/updateStatus — ohne das Formular zu öffnen. Das Formular muss aber vor der Zusage ausgefüllt werden.

Zusätzlich: Die actionStatusOptions-Konstante wird nur noch im Status-Submenü verwendet und kann nach dem Umbau entfallen (Labels direkt in die Buttons).

Änderungen

Datei: frontend/src/routes/app/events/[eventId]/+page.svelte

Status-Submenü (ca. Zeile 966):

Aktuell (vereinfacht):

{#if hoveredSubmenu === 'status'}
  <form method="POST" action="?/updateStatus" use:enhance>
    <input type="hidden" name="personId" value={personId} />
    {#each actionStatusOptions as opt (opt.value)}
      <button type="submit" name="status" value={opt.value}>{opt.label}</button>
    {/each}
  </form>
{/if}

Neu:

{#if hoveredSubmenu === 'status'}
  <form method="POST" action="?/updateStatus" use:enhance>
    <input type="hidden" name="personId" value={personId} />
    <button type="submit" name="status" value="invited">Eingeladen</button>
  </form>
  <button onclick={() => { closeDropdown(); openGuestForm(row as Guest); }}
    class="flex w-full px-2 py-1.5 text-xs hover:bg-muted">
    Antworten
  </button>
  <form method="POST" action="?/updateStatus" use:enhance>
    <input type="hidden" name="personId" value={personId} />
    <button type="submit" name="status" value="declined">Abgesagt</button>
  </form>
{/if}

actionStatusOptions entfernen (ca. Zeile 215):

- const actionStatusOptions = [
-   { value: 'invited', label: 'Eingeladen' },
-   { value: 'confirmed', label: 'Antworten' },
-   { value: 'declined', label: 'Abgesagt' },
- ];

Bonus: Prüfen ob openGuestForm auch im Antworten-Tab (?tab=antworten) die "Beantworten"/"Ändern"-Buttons ersetzen sollte, damit der gesamte confirmed-Flow nur noch über das Modal läuft.

Betroffene Dateien

Datei Änderung
frontend/src/routes/app/events/[eventId]/+page.svelte Status-Submenü aufteilen, actionStatusOptions entfernen

Tests

  • events.test.ts: Test hat Dropdown-Menue in Aktionen-Spalte prüft auf closeDropdown und hoveredSubmenu — bleibt gültig
  • Prüfen ob Tests auf actionStatusOptions referenzieren (aktuell nein)

Commit-Konvention

feat(frontend): "Antworten" im Gaesteliste-Dropdown oeffnet Modal

- Status-Submenue: 'Antworten' als eigener Button mit openGuestForm
- Eingeladen/Abgesagt bleiben als direkte Status-Setzer
- actionStatusOptions entfernt (Labels jetzt hardcoded)
### Problem Im Gästeliste-Dropdown (`?tab=guests`) gibt es das Submenü "Status". Der Eintrag "Antworten" setzt aktuell direkt `status=confirmed` via `?/updateStatus` — ohne das Formular zu öffnen. Das Formular muss aber vor der Zusage ausgefüllt werden. Zusätzlich: Die `actionStatusOptions`-Konstante wird nur noch im Status-Submenü verwendet und kann nach dem Umbau entfallen (Labels direkt in die Buttons). ### Änderungen **Datei:** `frontend/src/routes/app/events/[eventId]/+page.svelte` **Status-Submenü (ca. Zeile 966):** Aktuell (vereinfacht): ```svelte {#if hoveredSubmenu === 'status'} <form method="POST" action="?/updateStatus" use:enhance> <input type="hidden" name="personId" value={personId} /> {#each actionStatusOptions as opt (opt.value)} <button type="submit" name="status" value={opt.value}>{opt.label}</button> {/each} </form> {/if} ``` Neu: ```svelte {#if hoveredSubmenu === 'status'} <form method="POST" action="?/updateStatus" use:enhance> <input type="hidden" name="personId" value={personId} /> <button type="submit" name="status" value="invited">Eingeladen</button> </form> <button onclick={() => { closeDropdown(); openGuestForm(row as Guest); }} class="flex w-full px-2 py-1.5 text-xs hover:bg-muted"> Antworten </button> <form method="POST" action="?/updateStatus" use:enhance> <input type="hidden" name="personId" value={personId} /> <button type="submit" name="status" value="declined">Abgesagt</button> </form> {/if} ``` **`actionStatusOptions` entfernen** (ca. Zeile 215): ```diff - const actionStatusOptions = [ - { value: 'invited', label: 'Eingeladen' }, - { value: 'confirmed', label: 'Antworten' }, - { value: 'declined', label: 'Abgesagt' }, - ]; ``` **Bonus:** Prüfen ob `openGuestForm` auch im Antworten-Tab (`?tab=antworten`) die "Beantworten"/"Ändern"-Buttons ersetzen sollte, damit der gesamte confirmed-Flow nur noch über das Modal läuft. ### Betroffene Dateien | Datei | Änderung | |-------|----------| | `frontend/src/routes/app/events/[eventId]/+page.svelte` | Status-Submenü aufteilen, `actionStatusOptions` entfernen | ### Tests - `events.test.ts`: Test `hat Dropdown-Menue in Aktionen-Spalte` prüft auf `closeDropdown` und `hoveredSubmenu` — bleibt gültig - Prüfen ob Tests auf `actionStatusOptions` referenzieren (aktuell nein) ### Commit-Konvention ``` feat(frontend): "Antworten" im Gaesteliste-Dropdown oeffnet Modal - Status-Submenue: 'Antworten' als eigener Button mit openGuestForm - Eingeladen/Abgesagt bleiben als direkte Status-Setzer - actionStatusOptions entfernt (Labels jetzt hardcoded) ```
niboer commented 2026-07-01 16:38:56 +02:00
Author
Owner
Copy link

Implemented in c7de9e5

  • Status-Submenue aufgeteilt: Antworten oeffnet GuestFormModal, Eingeladen/Abgesagt bleiben direkte Status-Setzer
  • actionStatusOptions entfernt
Implemented in c7de9e5 - Status-Submenue aufgeteilt: Antworten oeffnet GuestFormModal, Eingeladen/Abgesagt bleiben direkte Status-Setzer - actionStatusOptions entfernt
niboer commented 2026-07-01 16:52:45 +02:00
Author
Owner
Copy link

Follow-up in fa9d8f3: Unused confirm-Action auf Personenseite entfernt (direkter PUT status=confirmed ohne formFinal war nie im Template verdrahtet).

Follow-up in fa9d8f3: Unused confirm-Action auf Personenseite entfernt (direkter PUT status=confirmed ohne formFinal war nie im Template verdrahtet).
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
v0.5.0
v0.4.0
v0.3.0
v0.2.2
v0.2.1
v0.2.0
v0.1.0
Labels
Clear labels   
component/backend
Backend-Komponente

  
component/docs
Dokumentation (README, design.md, AGENTS.md)

  
component/e2e
E2E-Testinfrastruktur (Playwright, Docker-E2E-Runner)

  
component/frontend
Frontend (Cloudflare Worker, BFF, UI)

  
component/infra
Infrastruktur (Docker, CI/CD, Build)

  
component/keycloak
Keycloak (Realm, Theme, Admin API)

  
prio/high
Hohe Prioritaet (blockierend/kritisch)

  
prio/low
Niedrige Prioritaet

  
prio/medium
Mittlere Prioritaet

  
type/bug
Fehler oder unerwartetes Verhalten

  
type/chore
Wartungs- und Konfigurationsaufgaben

  
type/feature
Neues Feature

  
type/refactor
Refactoring

  
type/test
Tests

No labels
component/backend
component/docs
component/e2e
component/frontend
component/infra
component/keycloak
prio/high
prio/low
prio/medium
type/bug
type/chore
type/feature
type/refactor
type/test
Milestone
Clear milestone
No items
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
WompSchmiede/FamilienFeierPlaner#158
No description provided.