Skip to content
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

Adding support for the SharePoint ACE APIs #6610

Conversation

GrahamMcMynn
Copy link
Contributor

@GrahamMcMynn GrahamMcMynn commented Mar 29, 2023

Description

SharePoint Framework is adding a new adaptive card experience (ACE) where it is powered by a Bot. In order to be able to implement the bot we are updating the BotBuilder Framework to natively support these API's to make it easier for end users to add a bot powered ACE.

fixes #6613

Specific Changes

We support 4 new BotInvoke activities:
"cardExtension/getCardView"
"cardExtension/getQuickView"
"cardExtension/getPropertyPaneConfiguration"
"cardExtension/setPropertyPaneConfiguration"

The SharePointActivityHandler in Microsoft.Bot.Builder.SharePoint is the base class for developing a bot powered ACE. Users of this base class will need to implement the following 4 methods:

OnSharePointTaskGetCardViewAsync
OnSharePointTaskGetQuickViewAsync
OnSharePointTaskGetPropertyPaneConfigurationAsync
OnSharePointTaskSetPropertyPaneConfigurationAsync

Testing

This was tested using a test app. I will write unit tests in a further iteration of this PR.

@GrahamMcMynn GrahamMcMynn requested a review from a team as a code owner March 29, 2023 16:48
/// <summary>
/// Initializes a new instance of the <see cref="QuickViewData"/> class.
/// </summary>
public QuickViewData()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should actually be an empty interface, because the data property can be an object with an arbitrary amount of properties, so long as the properties correspond to something in the Adaptive card body

@tracyboehrer tracyboehrer added the Automation: Parity with js The PR needs to be ported to JS label Oct 16, 2023
@GrahamMcMynn GrahamMcMynn marked this pull request as draft November 1, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automation: Parity with js The PR needs to be ported to JS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The bot builder framework should support SharePoint bot driven ACEs
4 participants