Skip to content

Skip the narration question on a computer with no microphone - #1327

Merged
davidmckayv merged 2 commits into
release/1.0from
david/rc2-no-mic-prompt
Nov 9, 2026
Merged

davidmckayv merged 2 commits into
release/1.0from
david/rc2-no-mic-prompt

Conversation

@davidmckayv

Copy link
Copy Markdown
Contributor

What changed

On a computer with no microphone, Start recording on Teach a Bot no longer opens "Narrate while you record?". It records without narration and leaves the narration choice unmade, so the first recording on a computer with a microphone still asks, exactly as before.

Why

#1314 made Teach a Bot say "No microphone found. Narration is off." and turned the Narration switch off on such a computer, but the first-time question still appeared, offering Record with narration as its primary answer. Seen on Windows with OpenBot 1.0 RC1.

The question is now gated in useRecordingWith (app/src/components/browser/recording.tsx) on the same useMicrophone() state #1314 added, so every place that records (Teach a Bot on Desktop and hosted, and a Bot's browser) behaves the same. A microphone that is present or not known still gets the question.

Test

app/tests/browser-page.test.tsx: in the OpenBot Desktop window, with the device-listing fake from #1314, a computer with a microphone is asked; after the microphone is unplugged, Start recording records at once with narrate: false, no question and no preference written. Fails on release/1.0 (no recording starts), passes with the fix.

Teach a Bot already says "No microphone found. Narration is off." and turns
the Narration switch off on a computer with no microphone, yet Start
recording still asked "Narrate while you record?" with Record with narration
as its first answer. Record now asks only when a microphone is there or not
known; with none it records without narration and leaves the choice unmade,
so the first recording with a microphone still asks.
@davidmckayv
davidmckayv force-pushed the david/rc2-no-mic-prompt branch from 34b6573 to fd294ac Compare November 9, 2026 02:17
@davidmckayv
davidmckayv merged commit 09dde85 into release/1.0 Nov 9, 2026
19 checks passed
@davidmckayv
davidmckayv deleted the david/rc2-no-mic-prompt branch November 9, 2026 02:42
davidmckayv added a commit that referenced this pull request Nov 9, 2026
Adds what landed on release/1.0 since rc.1 (#1297, #1298, #1302, #1324, #1326, #1327, #1328), rewrites #1322's entry to cover every waiting approval, and names the two new migrations where the notes said there were none.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant