janpeter/Media-Organizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: hervat slideshow na folderovergang #71

Merged
janpeter merged 6 commits from codex/fix-slideshow-resume into main 2026-07-31 22:40:52 +02:00
Conversation 1 Commits 6 Files changed 11 +961 -38
janpeter commented 2026-07-31 22:40:07 +02:00
Owner
Copy link

Samenvatting

  • houdt de slideshow open bij vorige/volgende folder
  • bewaart speel- of pauzestatus
  • navigeert depth-first door subfolders en slaat lege folders over
  • brengt de commits die na het mergen van PR #70 op de oude branch belandden alsnog naar main

Verificatie

  • npm test: 366/366 geslaagd
  • npm run build: geslaagd
  • gerichte ESLint: 0 fouten (1 bestaande waarschuwing in src/actions/storage.ts)
  • git diff --check: geslaagd
## Samenvatting - houdt de slideshow open bij vorige/volgende folder - bewaart speel- of pauzestatus - navigeert depth-first door subfolders en slaat lege folders over - brengt de commits die na het mergen van PR #70 op de oude branch belandden alsnog naar main ## Verificatie - npm test: 366/366 geslaagd - npm run build: geslaagd - gerichte ESLint: 0 fouten (1 bestaande waarschuwing in src/actions/storage.ts) - git diff --check: geslaagd
feat: keep hierarchical folder slideshows open
All checks were successful
CI / docker-build (pull_request) Successful in 1m18s
Details
d509e38afc
s4m-codex-reviewer approved these changes 2026-07-31 22:42:35 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

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

Findings

  • Geen blokkerende of error-severity findings gevonden.

Verificatie

  • npx tsx --test src/lib/folder-navigation.test.ts src/lib/slideshow-media.test.ts src/lib/slideshow-keyboard.test.ts src/lib/folder-browser.test.ts — PASS.
  • git diff --check origin/main...HEAD — PASS.
  • npm run build — niet volledig uitvoerbaar in tijdelijke review-clone doordat dependency next daar niet geïnstalleerd is (sh: 1: next: not found); dit is als omgevingsbeperking beschouwd, niet als PR-finding.
# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings gevonden. ## Verificatie - `npx tsx --test src/lib/folder-navigation.test.ts src/lib/slideshow-media.test.ts src/lib/slideshow-keyboard.test.ts src/lib/folder-browser.test.ts` — PASS. - `git diff --check origin/main...HEAD` — PASS. - `npm run build` — niet volledig uitvoerbaar in tijdelijke review-clone doordat dependency `next` daar niet geïnstalleerd is (`sh: 1: next: not found`); dit is als omgevingsbeperking beschouwd, niet als PR-finding.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
janpeter/Media-Organizer!71
No description provided.