IDEA-230: map met één klik naar prullenbak en selectief verwijderen #98
No reviewers
Labels
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
janpeter/Media-Organizer!98
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "codex/idea-230-folder-trash"
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?
Samenvatting
Verificatie op commit
59b45561bbee1bc9e724975f42223d6c8479937anpm testmet de allowlisted lokale PostgreSQL-testdatabase en TAP-reporter: 1.174 tests, 1.143 geslaagd, 0 fouten, 31 geconfigureerde skips.npm run typecheck,npm run build: geslaagd.npm run lint: 0 fouten, 11 bestaande waarschuwingen.Herkomst en grens
IDEA-230 / sprint S-2026-09-29-1 / PBI-32 / T-218. Implementatieplan in deze PR; het PBI is gekoppeld aan ProductDoc-planrevisie
cmumqdwlj00032i179zz0afxw. De bijgewerkte mapspecificatie staat als nieuwe ProductDoc-revisiecmumtssu9001m2i171uobfyhj; de eerdere dubbele GO geldt alleen voor de oude, gepinde revisie. Geen merge, migratie, productieactivering of nieuwe Linux/NFS-productieproef is in deze PR uitgevoerd. Het bestaandedocs/runbooks/folder-trash-deploy.mdblijft de uitrolgrens.De lokale gates zijn groen op
59b4556; de Forgejo-checks voor deze head lopen nog. Op de vorige head waren alle vijf checks groen. De afzonderlijkeempty-trash-job slaagde toen na 25m43s; de runnerduur varieerde tussen CI-runs. De gesloten 10 seconden database-timeout en de 10.000-doelenproef zijn niet versoepeld. Merge en productie-uitrol vragen een afzonderlijk besluit.Verdict: APPROVED
Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
COMMENT
src/components/DiskDeleteControls.tsx:38: individuele selectievakjes begrenzen de selectie niet op 100. Daardoor kan een gebruiker op een pagina met meer dan 100 bestanden meer dan het contractuele maximum selecteren en pas na de onomkeerbare bevestiging een generieke serverafwijzing krijgen. Blokkeer/selecteer niet verder boven de limiet (en geef die limiet direct in de UI aan).Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
REQUEST_CHANGES
Findings
src/components/useFolderTrashToggle.ts:127: polling kijkt uitsluitend naar lokaleinventory-state. Na een paginaverversing of terugnavigeren komt een nog lopende actie alleen viaitem.trashAction.status === "inventory"binnen; die wordt inlocalniet geïnitialiseerd. Daardoor start geen statusread/poll, blijft de knop onbeperkt “Status vernieuwen/Bezig” en wordt hij niet automatisch rood zodra de inventarisatie duurzaam publiceert. Dit schendt de vereiste interactie dat de knop tijdens verzamelen bezet is en pas na publicatie rood wordt. Initialiseer/poll ook server-afkomstige inventory-acties (zonder nieuwe write) en dek de reload-route met een test.Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Verdict: APPROVED
Findings
Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
De selectie voor definitieve verwijdering blijft server-geautoriseerd (actie, directe ouder, entry-status/generatie en bronbinding worden onder de mountlock hertoetst); de UI voorkomt dubbel verzenden en herhaalt geen onzekere schrijfuitkomsten. De implementatie volgt de mapprullenbak-specificatie en de MD3-tokenconventies; gerichte DB-, actie- en mounted UI-tests zijn met de wijziging uitgebreid.