-
Notifications
You must be signed in to change notification settings - Fork 0
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
Test and update the intents in the JavaScript code #3
Comments
@UditArora2000 Each live end-to-end test should ensure the following requirements:
Conclusion: If a test seems not to be possible, then probably an intent is mussing that would be required anyways to improve the explainability. Hence, implementing these tests will have a two-fold impact: We check the quality and extend the explainability. |
@Perevalov Hi, I am a little confused because there is no intent on Dialogflow named - |
Hi, at first check whether it is present in the previous year's version |
@UditArora2000 Let's stop this task to follow a more productive path. |
@Perevalov @anbo-de I was working on making the functions, but I am stuck on the following - when I need to fetch the components from qanary (link), I need to wait till all components are fetched, but this all happens inside a request to the |
@UditArora2000 What's the point of making |
@Perevalov Sure, then I'll proceed with making it a sync function. |
@anbo-de Could you provide what the role of these intents is in regards to their implementation on Dialogflow:
Additionally, what is the difference between the intents - |
@UditArora2000 In the following, I will define the behavior as I understand it.
IMO the purpose is the same. As far as I remember, our earlier intention was to distinguish the intents between asking from the component perspective and the profile perspective.
|
Context
For each component, implement a unit test by calling the Dialogflow API and activating the corresponding intent. The response result has to correspond the expected result within the context of the intent.
DoD
The text was updated successfully, but these errors were encountered: