From cedc1380dd5017e6b87e834c604ef39befaa4593 Mon Sep 17 00:00:00 2001 From: "matthias.lotz" Date: Thu, 13 Nov 2025 20:23:04 +0100 Subject: [PATCH] docs: Update FEATURE_PLAN with Issue 6 & 7 resolution --- docs/FEATURE_PLAN-social-media.md | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/docs/FEATURE_PLAN-social-media.md b/docs/FEATURE_PLAN-social-media.md index 759c8f3..17f2487 100644 --- a/docs/FEATURE_PLAN-social-media.md +++ b/docs/FEATURE_PLAN-social-media.md @@ -1109,15 +1109,26 @@ MANAGEMENT_TOKEN_EXPIRY=90 **Commit**: `8e62475` - "fix: DatabaseManager removes inline comments correctly in migrations" **Test**: Migration 005 & 006 laufen jetzt automatisch beim Backend-Start ✅ -### Issue 6: ModerationGroupsPage - Filter "Alle Gruppen" (13. Nov) - ⚠️ OFFEN -**Problem**: Filter "Alle Gruppen" auf ModerationGroupsPage.js funktioniert nicht (mehr?) -**Status**: Neu entdeckt während Testing von Tasks 12 & 20 -**Next**: Separate Bugfix-Session nach Commit von Tasks 12 & 20 +### Issue 6: ModerationGroupsPage - Filter "Alle Gruppen" (13. Nov) - ✅ GELÖST +**Problem**: Filter "Alle Gruppen" auf ModerationGroupsPage.js zeigte nicht alle Gruppen +**Ursache**: Backend filterte Gruppen mit `display_in_workshop=1` auch wenn kein Filter gesetzt war +**Lösung**: Filter-Bedingung im else-Block entfernt - zeigt jetzt wirklich ALLE Gruppen +**Commit**: `58a5c95` - "fix(phase2): Fix API routes and filter logic (Issues 6 & 7)" +**Test**: GET /moderation/groups liefert jetzt 73 Gruppen (alle) -### Issue 7: Export-Button funktioniert nicht (13. Nov) - ⚠️ OFFEN -**Problem**: "Consent-Daten exportieren" Button funktioniert nicht (mehr?) -**Status**: Neu entdeckt während Testing von Tasks 12 & 20 -**Next**: Separate Bugfix-Session nach Commit von Tasks 12 & 20 +### Issue 7: Export-Button funktioniert nicht (13. Nov) - ✅ GELÖST +**Problem**: "Consent-Daten exportieren" Button funktionierte nicht +**Ursache**: Routes hatten falschen Pfad-Prefix (`/admin/*` statt `/api/admin/*`) +**Lösung**: `/api` Prefix zu Consent-Admin-Routes hinzugefügt für Konsistenz +**Betroffene Routes**: + - GET `/api/admin/groups/by-consent` (vorher: `/admin/groups/by-consent`) + - GET `/api/admin/consents/export` (vorher: `/admin/consents/export`) +**Commit**: `58a5c95` - "fix(phase2): Fix API routes and filter logic (Issues 6 & 7)" +**Test**: + - ✅ CSV-Export funktioniert: `curl http://localhost:5001/api/admin/consents/export?format=csv` + - ✅ Dynamische Platform-Spalten: facebook, instagram, tiktok + - ✅ Test-Upload mit Social Media Consents erfolgreich + - ✅ Export zeigt zugestimmte Plattformen pro Gruppe ## 📊 Implementierungsergebnis