FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[QnA] Fix MultiTurnPrompt QnA conversation by Batta32 · Pull Request #1204 · microsoft/botbuilder-java · GitHub

This repository was archived by the owner on Dec 4, 2023. It is now read-only.
/ botbuilder-java Public archive

[QnA] Fix MultiTurnPrompt QnA conversation - #1204

Merged
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
southworks:external/fix/southworks/java/qna/qnamakerdialog
May 26, 2021
Merged

[QnA] Fix MultiTurnPrompt QnA conversation#1204
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
southworks:external/fix/southworks/java/qna/qnamakerdialog

Conversation

Martin Battaglino (Batta32) commented May 11, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes #1203
Related to #1166

Description

When the MultiTurnPrompt QnA conversation is configured and executed in a Java sample (e.g. 49.qnamaker-all-features), the conversation has an infinite loop.

Specific Changes

  • Get the response from the qnaMakerClient (like JS)
  • Obtain a default empty dictionary when the QNA_CONTEXT_DATA is not present in the state of the active dialog (like C#)

Testing

MultiTurnPrompt working as expected

mvn clean install passing successfully

Copy link
Copy Markdown
Contributor Author

tracyboehrer (@tracyboehrer) and Lee Parrish (@LeeParrishMSFT), we updated this branch adding the last changes of the main branch and it's ready to be reviewed and merged 😊!

tracyboehrer (tracyboehrer) merged commit 955b836 into microsoft:main May 26, 2021
Martin Battaglino (Batta32) deleted the external/fix/southworks/java/qna/qnamakerdialog branch May 26, 2021 15:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[QnA] Infinite loop using MultiTurnPrompt

2 participants


Back | FazBrowse Home | New Git URL