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

[Docs] Events and Triggers Topic #1394

Closed
jameslew opened this issue Oct 30, 2019 · 4 comments
Closed

[Docs] Events and Triggers Topic #1394

jameslew opened this issue Oct 30, 2019 · 4 comments

Comments

@jameslew
Copy link

https://github.com/microsoft/BotFramework-Composer/blob/kaiqb/Ignite2019/docs/concept-events-and-triggers-draft.md#handle-intent-events

  1. "The Handle Unknown Intent handles "none" intent." - does this mean specifically the LUIS none intent, or any unknown intent?

https://github.com/microsoft/BotFramework-Composer/blob/kaiqb/Ignite2019/docs/concept-events-and-triggers-draft.md#luis-recognizer

  1. "The content is managed by LUDown, a command line tool that helps covert .lu file(s) into JSON files for language processing applications such as LUIS and QnA knowledge base." - not sure customer needs to know about LUDown (or that it even exists). Also we don't feed QnA anymore from LU files, correct? It's a new format or something? Also, consider using "language understanding"

  2. "child dialogs using the syntax memory path" - what is a syntax memory path? Link?

  3. "On the right side of the Composer menu, choose Regular Expression as recognizer type" - include "for your trigger" as that is a context sensitive panel?

@jameslew jameslew added the Needs-triage A new issue that require triage label Oct 30, 2019
@zxyanliu
Copy link
Contributor

Thanks for your feedback.
I have fixed 2 and 4 and I have reached out to @benbrown to confirm 1 and 3.

@Kaiqb Kaiqb added Public Preview and removed Needs-triage A new issue that require triage labels Oct 31, 2019
@vishwacsena
Copy link
Contributor

For #1, it's any intent that is not handled by a trigger. See here

For #3, memory scopes are documented here

@zxyanliu
Copy link
Contributor

Fixed all. See here.

@zxyanliu
Copy link
Contributor

zxyanliu commented Nov 7, 2019

Closing since all bugs are fixed.

@zxyanliu zxyanliu closed this as completed Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants