Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Conversation

@forslund
Copy link
Collaborator

==== Tech Notes ====

  • Mycroft did not acknowledge that he was woken due to missing forwarder
    of speak commands to message bus.
  • Removed speak when audio could not be understood

@forslund forslund added CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) tagged for review and potential merge labels Sep 15, 2017
@forslund forslund requested a review from penrods September 15, 2017 10:39
====  Tech Notes ====
Mycroft did not acknowledge that he was woken due to missing forwarder
of speak commands to message bus.
Copy link
Contributor

@penrods penrods left a comment

Choose a reason for hiding this comment

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

LGTM

@penrods penrods merged commit 1b50c02 into MycroftAI:dev Sep 15, 2017
@forslund forslund deleted the bugfix/i-am-awake branch September 22, 2017 08:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants