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

Feature/issue 74 #75

Open
wants to merge 5 commits into
base: 20.02
Choose a base branch
from
Open

Conversation

wamsachel
Copy link

WIP first draft updating timer skill

To use:

@wamsachel
Copy link
Author

WIP first draft updating timer skill

Example usage:

  • Set a timer for every 60 seconds
  • ping me every 30 minutes
  • create an interval timer for every 10 seconds

@krisgesling
Copy link
Contributor

krisgesling commented Oct 8, 2020

Hey there, I just had a quick look at the changes so far, and I wanted to make sure the difference between Adapt and Padatious intents was clear, you can read more here:
https://mycroft-ai.gitbook.io/docs/skill-development/user-interaction/intents

It looks like you're using Padatious .intent files so the @intent_handler decorator should probably be something like:

@intent_handler('start.interval.timer.intent')

If you have any questions about Skill development, the ~Skills Channel on Mycroft Chat is a great place to find other devs.

@wamsachel
Copy link
Author

Thanks for the feedback Kris!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants