-
Notifications
You must be signed in to change notification settings - Fork 492
Update TeamsInfo to support CloudAdapter #5728
Conversation
| /// <summary> | ||
| /// Creates a new thread in a team chat and sends an activity to that new thread. | ||
| /// </summary> | ||
| /// <param name="turnContext"> Turn context. </param> |
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.
nit: spacing looks odd before/after the param comments here
EricDahlvang
left a comment
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 might need to revisit this when we add support for create conversation from a skill. This will probably require audience.
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.
I think the descriptions got mixed up - the description for activity is the channel ID, and channel ID has a fragment of "cancellation token." (in the code comments)
|
Thanks @clearab nice catch. That must have been there for ages - I updated the old functions description too. |
Pull Request Test Coverage Report for Build 254556
💛 - Coveralls |
mrivera-ms
left a comment
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.
🕐
mrivera-ms
left a comment
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.
![]()
fixes #5685