Skip to content

Fix microphone button not re-enabled after error#2241

Merged
corinagum merged 5 commits intomicrosoft:masterfrom
compulim:fix-2240
Jul 31, 2019
Merged

Fix microphone button not re-enabled after error#2241
corinagum merged 5 commits intomicrosoft:masterfrom
compulim:fix-2240

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Jul 30, 2019

Fixes #2240.

Changelog Entry

Description

Microphone button is permanently disabled after receiving and error from speech engine. The error could be:

  • Muted microphone
  • No recognizable speech
  • Network error

Instead, the microphone button should not be permanently disabled.

Specific Changes

  • Set dictateState to IDLE even if currently is STOPPING

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 30, 2019

Coverage Status

Coverage remained the same at 64.984% when pulling 7c26df0 on compulim:fix-2240 into 5dc8e43 on microsoft:master.

Copy link
Copy Markdown
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@corinagum corinagum merged commit 76c0f20 into microsoft:master Jul 31, 2019
@compulim compulim mentioned this pull request Oct 4, 2019
13 tasks
@compulim compulim mentioned this pull request Oct 25, 2019
55 tasks
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.

Speech: Stop microphone button should re-enable the button after a few seconds

3 participants