fix: herstel foldernavigatie in LibraryFolderView-slideshows #70

Merged

janpeter merged 4 commits from codex/fix-folder-slideshow-navigation into main

2026-07-31 21:44:10 +02:00

janpeter commented

2026-07-31 21:37:58 +02:00

Owner

Samenvatting

voeg expliciete currentSlug en callbacknavigatie toe aan SlideshowFolderAlbums
geef folderAlbums via MediaGrid door
leid de parent en siblingfolders af in LibraryFolderView
behoud de bestaande href-navigatie voor de vijf librarypagina’s

Verificatie

npm test: 361/361 groen
npm run build: geslaagd
git diff --check: schoon

Scrum4Me: PBI-14, ST-023, T-115 t/m T-117.

## Samenvatting - voeg expliciete currentSlug en callbacknavigatie toe aan SlideshowFolderAlbums - geef folderAlbums via MediaGrid door - leid de parent en siblingfolders af in LibraryFolderView - behoud de bestaande href-navigatie voor de vijf librarypagina’s ## Verificatie - npm test: 361/361 groen - npm run build: geslaagd - git diff --check: schoon Scrum4Me: PBI-14, ST-023, T-115 t/m T-117.

janpeter added 4 commits

2026-07-31 21:37:58 +02:00

fix: derive folder browser slideshow targets f01c208d78

fix: support callback folder navigation in slideshow ab0683ecbc

fix: provide folder siblings to media slideshow a3cf593892

docs: record folder slideshow navigation plan

CI / docker-build (pull_request) Successful in 1m20s

Details

05400ec7ae

s4m-codex-reviewer approved these changes

2026-07-31 21:42:44 +02:00

s4m-codex-reviewer left a comment

Verdict: APPROVED

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

Geen blokkerende of error-severity findings gevonden.

Review-notities

De wijziging volgt de bestaande folder-browser/slideshow architectuur: virtuele folder-node-id's blijven strings, MediaGrid transporteert alleen configuratie, en de bestaande href-navigatie blijft behouden wanneer geen callback is meegegeven.
Tests toegevoegd voor parentFolderNodeId en de nieuwe slideshow target-helper dekken zowel callbacknavigatie als bestaande URL-navigatie af.
Verificatie: npx tsx --test src/lib/folder-browser.test.ts src/lib/slideshow-media.test.ts src/lib/slideshow-keyboard.test.ts slaagde lokaal in de tijdelijke clone na herstel van install-permissies.
npm run build kon in deze tijdelijke clone niet volledig worden afgerond door omgevings-/repo-setupvereisten buiten de diff (@s4m-kit/index, ontbrekende gegenereerde Prisma client, native Next/Tailwind bindings). Ik heb geen aanwijzing gevonden dat die fouten door deze PR zijn geïntroduceerd.

# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings gevonden. ## Review-notities - De wijziging volgt de bestaande folder-browser/slideshow architectuur: virtuele folder-node-id's blijven strings, `MediaGrid` transporteert alleen configuratie, en de bestaande href-navigatie blijft behouden wanneer geen callback is meegegeven. - Tests toegevoegd voor `parentFolderNodeId` en de nieuwe slideshow target-helper dekken zowel callbacknavigatie als bestaande URL-navigatie af. - Verificatie: `npx tsx --test src/lib/folder-browser.test.ts src/lib/slideshow-media.test.ts src/lib/slideshow-keyboard.test.ts` slaagde lokaal in de tijdelijke clone na herstel van install-permissies. - `npm run build` kon in deze tijdelijke clone niet volledig worden afgerond door omgevings-/repo-setupvereisten buiten de diff (`@s4m-kit/index`, ontbrekende gegenereerde Prisma client, native Next/Tailwind bindings). Ik heb geen aanwijzing gevonden dat die fouten door deze PR zijn geïntroduceerd.

janpeter merged commit d1cbf8d710 into main

2026-07-31 21:44:10 +02:00

janpeter referenced this pull request from a commit

2026-07-31 21:44:12 +02:00

Merge pull request 'fix: herstel foldernavigatie in LibraryFolderView-slideshows' (#70) from codex/fix-folder-slideshow-navigation into main

janpeter referenced this pull request

2026-07-31 22:40:07 +02:00

fix: hervat slideshow na folderovergang #71