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

Add "trigger" support to respond to non-command text #117

Open
clockworksoul opened this issue Oct 13, 2021 · 2 comments
Open

Add "trigger" support to respond to non-command text #117

clockworksoul opened this issue Oct 13, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request needs-docs This issue will need addition or changes to documentation
Milestone

Comments

@clockworksoul
Copy link
Member

Currently Gort only responds to explicit commands, formatted like structured commands. There should be some kind of ability to respond to non-command text, sort of like Slack "triggers" (but not like Cog "triggers", which are very different).

Triggers would be configured with a pattern to match and a command, so when Gort sees text matching the defined pattern, it executes the command.

  • Open question: How will it behave wrt to trigger text + command inputs?

    • Should input be passed through to the command parser? Maybe set as an envvar?
  • What if a command also matches a trigger? Which takes precedence?

    • Execute triggers only if there’s no matching command?
    • Execute trigger and stop processing (so command never executes)?
    • Either of the above/configurable per trigger?
  • What about timed triggers (maybe use cron definitions)?

@clockworksoul clockworksoul added the enhancement New feature or request label Oct 13, 2021
@clockworksoul clockworksoul changed the title Add "trigger" support to do things in response to non-command text Add "trigger" support, to do things in response to non-command text Oct 13, 2021
@clockworksoul clockworksoul changed the title Add "trigger" support, to do things in response to non-command text Add "trigger" support to respond to non-command text Oct 13, 2021
@theothertomelliott theothertomelliott self-assigned this Dec 5, 2021
@clockworksoul clockworksoul added this to the 0.9.3 milestone Jan 4, 2022
@theothertomelliott
Copy link
Contributor

@clockworksoul Given the above commit can this be closed? Or was there follow up work needed?

@theothertomelliott theothertomelliott added the needs-docs This issue will need addition or changes to documentation label Jun 6, 2022
@theothertomelliott
Copy link
Contributor

As discussed "in-person" this will need documenting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-docs This issue will need addition or changes to documentation
Projects
None yet
Development

No branches or pull requests

2 participants