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

Extracted QnAMaker logic from corresponding middleware #148

Merged
merged 1 commit into from
Feb 18, 2018
Merged

Extracted QnAMaker logic from corresponding middleware #148

merged 1 commit into from
Feb 18, 2018

Conversation

lkts
Copy link
Contributor

@lkts lkts commented Feb 17, 2018

Moved GetAnswers logic to separate class that is used in middleware and can be reused without it. There are test for QnA but they are commented and i don`t know how to run them.

@drub0y @daltskin

Closes #124 and i believe #137 as well.

@msftclas
Copy link

msftclas commented Feb 17, 2018

CLA assistant check
All CLA requirements met.

@cleemullins
Copy link
Contributor

Thanks! I’ll take a look through this later, and make sure the tests pass.

@cleemullins cleemullins merged commit 562f029 into microsoft:master Feb 18, 2018
@cleemullins
Copy link
Contributor

Thanks!!!!

@lkts lkts deleted the qnamaker-extract branch February 18, 2018 07:46
ceciliaavila pushed a commit that referenced this pull request Jul 1, 2019
gasper-az pushed a commit that referenced this pull request Aug 15, 2019
ceciliaavila pushed a commit that referenced this pull request Sep 5, 2019
ShYuPe pushed a commit to ShYuPe/botbuilder-dotnet that referenced this pull request Aug 25, 2020
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.

3 participants