-
Notifications
You must be signed in to change notification settings - Fork 479
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
issue 1960 private to public #2117
Conversation
johnataylor
commented
Jun 24, 2019
- make DialogContext constructor public (it was internal) this was a customer ask from two independent sources
- make DelegatingTurnContext protected rather than private in ActivityHandler - this allows derived classes to also get the benefit of slightly stronger typing (this was really a bug as ActivityHandler was intended for extensibility through inheritance)
Pull Request Test Coverage Report for Build 66903
💛 - Coveralls |
❌ 1 Binary Compatibility issues for Microsoft.Bot.Builder compared against version 4.3.1
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1 |
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.
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.
Looks good
This was reverted. I also "undeleted" the branch so it can be worked on. |