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

speech plugin is always enabled on dev #3682

Closed
PieterGit opened this issue Jul 10, 2018 · 5 comments
Closed

speech plugin is always enabled on dev #3682

PieterGit opened this issue Jul 10, 2018 · 5 comments
Assignees
Labels
bug regression regression, used to work in previous released version
Milestone

Comments

@PieterGit
Copy link
Contributor

Without adding speech to ENABLE I get speech messages with current dev.

I think it's caused by these lines:

var speechPlugin = client.plugins('speech');

I think the server settings must be checked whether the speech plugin is enabled.

@PieterGit PieterGit added bug regression regression, used to work in previous released version labels Jul 10, 2018
@PieterGit PieterGit added this to the 0.10.3 milestone Jul 10, 2018
@PieterGit
Copy link
Contributor Author

PieterGit commented Jul 11, 2018

temporary fix added to dev. this will disable the complete plugin until @sulkaharo has the time to add the proper code to only execute it when the plugin is enabled.

@sulkaharo
Copy link
Member

Did it speak out even if you checked off the box in the client settings?

@PieterGit
Copy link
Contributor Author

@sulkaharo we didn't add speech to the ENABLE var and then it would speak it out on page load.
If I selected a Dutch anguage it spoke the English text with a dutch accent.
I could find a check box in the client settings (hamburger menu), even if I enable the speech plugin.
Are you sure you committed/pushed all the changes? 😄

@jaylagorio
Copy link
Contributor

Can you also make this observe the ?mute=true URL parameter so if that's set it doesn't talk?

@PieterGit
Copy link
Contributor Author

@sulkaharo fixed this with d80c605
@jaylagorio could you add add separate feature request for the ?mute=true URL parameter? This issue was about the fact that it played stuff even, when you didn't have the plugin enabled.

Closing this issue because it's fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug regression regression, used to work in previous released version
Projects
None yet
Development

No branches or pull requests

3 participants