-
Notifications
You must be signed in to change notification settings - Fork 529
[C#][Bot-Solutions] Rehome used services of Microsoft.Bot.Configuration to Microsoft.Bot.Solutions in Services folder #3618
Conversation
Hey @Batta32, I'm not sure the right path forward on this one is to mark these properties as obsolete. I'm assuming we are still ultimately need these in the current Virtual Assistant. If there's an alternative to LuisService that we are intended to be using, then I think we should figure out what that is. Let's discuss and we can figure out a path forward with the SDK team. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's talk to the SDK team first - would like to understand if there is an intended alternative to LuisService if it's been marked as deprecated in the SDK.
@peterinnesmsft - Sure! At the meantime, I'll change this PR to draft and we will review another alternatives with the team 😊. |
@peterinnesmsft - we finished doing the rehome of the used services of |
Fixes #3617
Purpose
What is the context of this pull request? Why is it being done?
With the new Microsoft.Bot.Builder@4.10.0 packages, the LuisService class has been deprecated so Bot-Solutions doesn't build as it runs into a "LuisService is obsolete: This class is deprecated" error.
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
Rehome the used Services of Microsoft.Bot.Configuration to Microsoft.Bot.Solutions:
Tests
Is this covered by existing tests or new ones? If no, why not?
We tested the C# solutions, after apply this fix, manually and worked as expected.
Successfully communication in Emulator
![image](https://user-images.githubusercontent.com/37625424/90182840-eece2700-dd88-11ea-96f4-93e89f6fea72.png)
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
-
Checklist
General