hobbyhimmel/Project-Image-Uploader
8
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs: Update FEATURE_PLAN with Issue 6 & 7 resolution

Browse Source
This commit is contained in:
Matthias Lotz 2025-11-13 20:23:04 +01:00
parent 58a5c95d42
commit cedc1380dd
1 changed files with 19 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
docs/FEATURE_PLAN-social-media.md
View File

@ -1109,15 +1109,26 @@ MANAGEMENT_TOKEN_EXPIRY=90
**Commit**: `8e62475` - "fix: DatabaseManager removes inline comments correctly in migrations" **Commit**: `8e62475` - "fix: DatabaseManager removes inline comments correctly in migrations"
**Test**: Migration 005 & 006 laufen jetzt automatisch beim Backend-Start ✅ **Test**: Migration 005 & 006 laufen jetzt automatisch beim Backend-Start ✅
### Issue 6: ModerationGroupsPage - Filter "Alle Gruppen" (13. Nov) - ⚠️ OFFEN ### Issue 6: ModerationGroupsPage - Filter "Alle Gruppen" (13. Nov) - ✅ GELÖST
**Problem**: Filter "Alle Gruppen" auf ModerationGroupsPage.js funktioniert nicht (mehr?) **Problem**: Filter "Alle Gruppen" auf ModerationGroupsPage.js zeigte nicht alle Gruppen
**Status**: Neu entdeckt während Testing von Tasks 12 & 20 **Ursache**: Backend filterte Gruppen mit `display_in_workshop=1` auch wenn kein Filter gesetzt war
**Next**: Separate Bugfix-Session nach Commit von Tasks 12 & 20 **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 ### Issue 7: Export-Button funktioniert nicht (13. Nov) - ✅ GELÖST
**Problem**: "Consent-Daten exportieren" Button funktioniert nicht (mehr?) **Problem**: "Consent-Daten exportieren" Button funktionierte nicht
**Status**: Neu entdeckt während Testing von Tasks 12 & 20 **Ursache**: Routes hatten falschen Pfad-Prefix (`/admin/*` statt `/api/admin/*`)
**Next**: Separate Bugfix-Session nach Commit von Tasks 12 & 20 **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 ## 📊 Implementierungsergebnis