-
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
[Botkit] Migrate Webex Adapter #2238
Conversation
1ddbbe0
to
6234889
Compare
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
1 similar comment
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
2080619
to
d270d90
Compare
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
d270d90
to
b7ae0ad
Compare
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
b7ae0ad
to
f458185
Compare
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
4 similar comments
✔️ 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 |
✔️ 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 |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
1 similar comment
✔️ 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 |
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.
We should add the new functionality from Webex, which includes:
- a new event type for receiving adaptive card submissions
- a secondary webhook subscription to request these events
- a new API call to decrypt these events
- a slightly different mapping of these events into Activities
This is all present in the current botbuilder-adapter-webex
libraries/Adapters/Microsoft.Bot.Builder.Adapters.Webex/WebexAdapter.cs
Outdated
Show resolved
Hide resolved
I think if possible we should expose more of the webex client as a public member of the adapter so that developers will have automatic access to all of the webex apis. The APIs used within the adapter are important, but developers will frequently need others and if we provide it via the adapter they don't have to create another instance of the same class. |
Here is the diff from when I added these new features to Botkit: |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
3ee02ab
to
2dbc83f
Compare
Yes, it sounds good. Once we finished the migration of the Adaptive Cards functionality, we can take this task. |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
c9d598a
to
1414224
Compare
✔️ 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 |
✔️ 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 |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1 |
Mergning, now that @benbrown has signed off via email.
|
Proposed Changes
Migrated the Webex Adapter from the Botkit repo.
Details
Migrated classes:
Helper classes: