Fix #83: Prev Records - ReferenceError durationStr out of scope #87

Merged
SaschaFuksa merged 1 commit from batch/83 into master 2026-04-22 19:21:21 +00:00
Owner

Fixes #83

The loadPreviousRecordings map() callback used durationStr which was only defined in the outer scope. Since the inner map() creates a new closure scope, durationStr was undefined → caught as "Failed to load."

Fix: define folderDurationStr locally inside the map() callback.

Fixes #83 The loadPreviousRecordings map() callback used durationStr which was only defined in the outer scope. Since the inner map() creates a new closure scope, durationStr was undefined → caught as "Failed to load." Fix: define folderDurationStr locally inside the map() callback.
Fix #83: Prev Records - ReferenceError due to durationStr out of scope
Some checks failed
continuous-integration/dummy CI/Linter check passed
CI / lint (pull_request) Has been cancelled
CI / test (pull_request) Has been cancelled
b5a2e96314
The loadPreviousRecordings map() callback at line ~1850 used durationStr
which was only defined in the outer scope (line ~1404). Since the inner
map() creates a new scope, durationStr was undefined → caught as
'Failed to load.'

Fix: define folderDurationStr locally inside the map() callback using
the same format as the outer scope.
SaschaFuksa deleted branch batch/83 2026-04-22 19:21:21 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
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
Tasch/whisper-transcribe!87
No description provided.