-
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
Adds SkypeMiddleware to patch Skype mentions so RemoveMentionText works #2289
Conversation
…el mentions are handled. Added MentionWorkaroundForSkypeMiddleware to patch Skype mentions.
…Middleware and added tests.
Pull Request Test Coverage Report for Build 72156
💛 - Coveralls |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
… comments, and simplified the Mention_Skype test.
Hey, issues are fixed. CAn you review again? |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
libraries/Microsoft.Bot.Builder/SkypeMentionNormalizeMiddleware.cs
Outdated
Show resolved
Hide resolved
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
Fixes #2181 Added SkypeMiddleware to patch Skype mentions so that RemoveMentionText can properly removed mention text.
ActivityExtensions.cs
Reverts commit 4daa9b9, with the addition of a Trim() added. That commit did not work for other channel mention types.
Microsoft.Bot.Builder\SkypeMiddleware.cs
Added middleware class SkypeMiddleware to remove the tags from Mention.Text for Skype mentions. This allows RemoveMentionText to properly remove user names from Activity.Text