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

uischema: port uischema 'trigger' part to component schema files #5160

Merged
merged 2 commits into from
Feb 17, 2021

Conversation

yeze322
Copy link
Contributor

@yeze322 yeze322 commented Feb 4, 2021

Follows #4341
Follows #5099

Description

In #4341, we port the ui schema form part to runtime in order to support the customization of Composer Form Editor;
In #5099, we port the ui schema flow part to support the customization of Action UI;

in this PR, the trigger part is added to runtime as well to support the customization of Composer's Trigger creation wizard.

Refs ui.schema definition change in SDK repo: microsoft/botframework-sdk#6178

Affected SDK $kinds

  • OnActivity
  • OnBeginDialog
  • OnCancelDialog
  • OnChooseIntent
  • OnConversationUpdateActivity
  • OnDialogEvent
  • OnEndOfConversationActivity
  • OnError
  • OnEventActivity
  • OnHandoffActivity
  • OnIntent
  • OnInvokeActivity
  • OnMessageActivity
  • OnMessageDeleteActivity
  • OnMessageReactionActivity
  • OnMessageUpdateActivity
  • OnQnAMatch
  • OnRepromptDialog
  • OnTypingActivity
  • OnUnknownIntent

Example of 'trigger' uischema

Take Microsoft.OnMessageActivity as an example:

{
    "$schema": "https://schemas.botframework.com/schemas/ui/v1.0/ui.schema",
    "form": {...},
    "trigger": {
	"label": "Message received (Message received activity)",
	"order": 5.81,
	"submenu": "Activities"
    }
}

Composer will take this uischema and let OnMessageActivity appear under 'Activities'
image

@yeze322

This comment has been minimized.

@cwhitten cwhitten marked this pull request as ready for review February 10, 2021 01:13
@cwhitten cwhitten requested review from a team as code owners February 10, 2021 01:13
@yeze322
Copy link
Contributor Author

yeze322 commented Feb 10, 2021

We can merge this PR after microsoft/botframework-sdk#6178 checked in. @EricDahlvang thanks for the approval!

The PR in Composer for enabling new Teams Trigger doesn't need to be a dependency of this one. I will create another PR for Teams scenario specifically.

@yeze322 yeze322 merged commit c33d62d into microsoft:main Feb 17, 2021
EricDahlvang pushed a commit that referenced this pull request Feb 17, 2021
mrivera-ms pushed a commit that referenced this pull request Feb 19, 2021
* Update handoff action uischema to show in Composer action menu (#5231)

* uischema: port uischema 'trigger' part to component schema files (#5160)

* add trigger uischema

* update whitespaces

* update UT output uischema

* add Teams Actions menu uischema

* add trigger uischema for Teams Triggers

* check in merged uischema

Co-authored-by: Gary Pretty <gary@garypretty.co.uk>
EricDahlvang pushed a commit that referenced this pull request Apr 7, 2021
* Update handoff action uischema to show in Composer action menu (#5231)

* uischema: port uischema 'trigger' part to component schema files (#5160)

* add trigger uischema

* update whitespaces

* update UT output uischema

* add Teams Actions menu uischema

* add trigger uischema for Teams Triggers

* check in merged uischema

Co-authored-by: Gary Pretty <gary@garypretty.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants