Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should interrupt speech synthesis after microphone button is clicked #2429

Merged
merged 11 commits into from
Sep 30, 2019

Conversation

compulim
Copy link
Contributor

@compulim compulim commented Sep 26, 2019

Fixes #2428.

(Will add new tests)

Changelog Entry

Fixed

  • Fix #2428. Should interrupt speech synthesis after microphone button is clicked, by @compulim in PR #2429

Description

Today, the microphone button does not stop synthesizing. And since synthesizing is still ongoing, microphone will not turn on to prevent echo.

Tomorrow, the microphone button will stop synthesizing. And this should immediately turn on the microphone.

Specific Changes

  • Add a new saga to stop synthesis when the user want to open the microphone
  • Modify speech samples to use authorization token instead of secret

  • Testing Added

samples/06.g.select-voice/index.html Outdated Show resolved Hide resolved
samples/06.g.select-voice/README.md Show resolved Hide resolved
samples/06.f.hybrid-speech/index.html Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Sep 27, 2019

Coverage Status

Coverage increased (+0.007%) to 68.007% when pulling f126dee on compulim:fix-2428 into 7de0a84 on microsoft:master.

compulim and others added 3 commits September 30, 2019 13:01
Co-Authored-By: Corina <14900841+corinagum@users.noreply.github.com>
…html

Co-Authored-By: Corina <14900841+corinagum@users.noreply.github.com>
@compulim compulim merged commit 5164930 into microsoft:master Sep 30, 2019
@compulim compulim deleted the fix-2428 branch September 30, 2019 21:33
@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.

Should interrupt speech synthesis after microphone button is clicked
3 participants