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

Prettied it up, applied some linting, changed hereIntent and hereAction #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

roeintense
Copy link

  • hereIntent and hereAction no longer return a function, just the false or true depending on inputs.
  • hereIntent and hereAction now use String.prototype.match() by default, to allow regular expressions or specific strings.
  • Other small tweaks.

@kevinmarchese
Copy link

Hi Roeintense,

I’m using the botkit.Luis.middleware which works however the problem is it also sends any message type, like interactive_messages to Luis for processing as well. I’m wondering if there is any trick to stopping this from happening inside the middleware because if one of my dynamic dropdown menus are selected and if any of them at some point match another intent it would fire that intent from another function, at least I think it will, like a bye or greeting intent which I’m trying to avoid. The good thing is the threshold isn’t great enough yet. I’d appreciate any thoughts u may have or any tricks you might suggest. Thank you for your time an consideration.

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

Successfully merging this pull request may close these issues.

2 participants