-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
External enable/disable intent #860
Conversation
@JarbasAI thank you that's a cool feature, can you fix the tests so we can merge it :) |
rebased |
https://github.com/JarbasAl/mycroft_jarbas_utils needs this PR |
Will review tomorrow! |
Allow intents to be enabled or disabled from outside needed for example in my intent_layers helper class to control what intents are active
Could have sworn I posted a message earlier. Looks and works good. Only thing I'd like to change is the name of the messages. |
With the update it looks good. I'll rebase it and squash it into a single commit and get it merged in the morning. |
Merged via #1524, closing. |
Allow intents to be enabled or disabled from outside the skill itself
needed for example in my intent_layers helper class to control what intents are active