feat: update recognizer UI in form (#7972)

* fix: export display name and sort order for templates outside of Composer (#6936)

* Adding template sort order json

* Moving templates.json

* Cleaning up routing to no longer use old creation experience

* Removing unneeded fetchTemplate param

* Adding logic to attmpt to get template order and display name from git hub with shipped sort order as back up

* Adding ben and patrick as Composer dir code owners

* fixing unit test

* Making PR changes

* Fixing references with extra param

* Fix unused props lint error

* Remove unneeded fetch template call

* Fixing unit test

* Adding correct mocks for unit test

* Adding route to support older paths

* fixing unit test for new creation routing

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: Route to appsettings correctly (#6992)

* Appsettings update

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Remove unused event

Signed-off-by: Srinaath Ravichandran <[email protected]>

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: adjust mechanism used to determine if/when luis and qna teaching bubbles should pop out (#6973)

* adjust mechnaism used to determine if luis and qna bubbs should pop

* change name of variable

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: the format of hover info in LG editor (#6858)

* fix hover lg template display format

* add dependency in package json

* fix test case failed

* modify test

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* feature: new design for Connections setting pane (#6724)

* remodel Adapters section into new design

* better formatting

* fix table formatting

* update l10n strings

* Update AdapterSection.tsx

* Update AdapterSection.tsx

* Update AdapterSettings.test.tsx

* Update en-US.json

* Update en-US.json

* Update AdapterSettings.test.tsx

* address PR comments

Co-authored-by: Tony Anziano <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* Update query to grab qna template readme (#7021)

* fix: misc bug fixes in creation (#6950)

* Added hash to tab routing

* Fixing manage profile deep link

* Styling changes to remove double scroll bar in package manager

* Cleaning up style naming

Co-authored-by: Chris Whitten <[email protected]>

* feat: show topics in left nav (#6987)

* display topics in left nav

* only show external link icon on hover

* sort system topics at end of list

* show tooltip when hovering on pva topic

* show system topic icon

* collapse topic list with Enter

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* Runtime log for skills (#7030)

* fix abs (#7041)

* chore: update l10n files (#6920)

* update l10n files

* update l10n files

* feat: include pva topics by default (#7058)

* include pva topics by default when pulling from pva

* allow autocomplete on select dialog

* limit height of callout

* update tooltip copy

Co-authored-by: Tony Anziano <[email protected]>

* ci: use github-actions eslint formatter (#7015)

dont prefix output for CI

update github-actions formatter

* 7011 (#7018)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: homepage header layout (#6926)

* fix header layout

* occupy screen width and fill card with background

* change card title style

* update card styles, color, font

* update styles

* update image setting

* update background , icon

* use NeutralColors.gray160

* define constant width height

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>

* fix: Broken SVG image for QnA maker in provisioning (#7094)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix: pva deep link in new creation (#7100)

* removing unused commented code

* supporting client routing to defineConversation for PVA scenarios

* Ensuring runtimeType does not render as a creation dropdown for PVA scenarios

Co-authored-by: Chris Whitten <[email protected]>

* feat: new skill manifest creation experience (#6661)

* add skill select profile component

* manifest file cud  api

* add skill select page and publish logic

* folder rename

* add lu files writing logic

* add redirect to publishing page action

* change folder name

* add lu file parsing and intents filtering

* Support .lu and .qna for static files

* 'notification'

* manifest file name

* add skill add callers logic

* add manifest path to BotStructureFilesPatterns

* fix e2e

* notification

* add publish skill settings update

* pop auth

* add skill hostendpoint

* fix comments

* title change

* fix copy url wrong

* fix filtered lu files

* merge selectManifest & fetchManifest & description popup

* next bugfix

* bugfix

* merge main & popup err

* Fix endpoint not update issue

* Fix new adaptive runtime copy skill manifests to wwwroot folder

* skip step revert

* tooltip

* add back button & fix url color

* Update the logic of updating skill host endpoint

* callers set

* remove the physical file provider

* back button

* default select publish profie

* disabled generated & publish button when no profie

* url wrong

* default publish target

* callers error

* add create publish profile dialog

* comments

* callers

* lint

* lint fix

* lint ignore

* test case

* remove manifest folder from read patterns

* comments

* comments & conflict

* type

* bugfix

* Fix the logic of updating skill settings

* delete duplicate manifest url

* add profile popup logic

Co-authored-by: Qi Kang <[email protected]>
Co-authored-by: Long Jun <[email protected]>
Co-authored-by: Long Alan <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Soroush <[email protected]>

* feat: update the experience of adding a remote skill scenario (#6640)

* update add remote skill UI

* polish

* add trigger after skill created

* add action to on intent trigger after add remote skill

* import orchestrator after add remote skill

* add loading spinner

* fix bug

* add skill and alowCalled into appsettings

* fix skill identifier wrong in new action

* fix skill identifier

* remove allowedCaller when remove skill

* fix skill not added after add remote skill

* fix orchestractor modal not show

* remove trigger when remove skill, fix bugs

* polish UI

* add endpoint dropdown

* en-us

* filter duplicate msAppId in allowedCaller

* disable entity in lueditor

* polish

* fix some comments

* fix lint

* fix comments

* add prerelease

* fix test, fix endpoints bug, append intent to  multiLanguage lufile

* polish

* disable next if lueditor error

* fix lint

* fix multi language

* show warning msg if remote lufile get fail

* fix typecheck

* fix comments

* fix test

* fix comments

* fix test

* fix comments

* change sidebar back

* enus

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: provisioning resource groups typing partial existing name (#7103)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix issues with existing name stopping is new

* fix: multiple minor bugs in provision and creation (#7064)

* Make create from qna url modal blocking

* fix for bug in new tab view:
add botProjectData as dep to regenerate botStatusList

* Adding prevew tag to node tab in template selection view

* Fix error message for bot name

* fixing typo

Co-authored-by: Chris Whitten <[email protected]>

* fix: Update OnChooseIntent template to support Orchestrator (#7004)

* Genericizing OnChooseIntent

* Fix broken schema

* Allow more than 2 choices

* linter fixes

* trigger.ts not sending locale to worker

* Fix to normalize the names in lgUtil

* Remove all OnChooseIntent LG from common

* Rename parent template name to ChooseIntent

* Minor refactor

* fix linter issues

Co-authored-by: Chris Whitten <[email protected]>

* fix: remodel ProjectTree using ul/li tags (#7063)

* swap out details/summary for ul/li

* fix placement of items

* Update ExpandableNode.tsx

* fix layout issues and clicking behavior

* fix variables

* more fixes

* Update ExpandableNode.tsx

* Update ProjectTree.tsx

* Update ProjectTree.tsx

* remove stray comment

* Update expandableNode.test.tsx

* add new timeout to tests

* Update Breadcrumb.spec.ts

* post-merge fixes

* scope breadcrumb tests to inside breadcrumb

Co-authored-by: Andy Brown <[email protected]>

* feat: diable /hide items when in PVA bot context (#7119)

* feat: migrate E2E test with bot creation v2 (#7040)

* migrate e2e test with create bot v2

* cache created bot from template

* update

* update

* update and merge

* update

Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* fix: publish targets was overwrited during opening abs (#7137)

* fix open abs, publish targets overwrite

* fix the get alias issue

Co-authored-by: Lu Han <[email protected]>

* fix: Remove tag entity menu from LU Toolbar (#7171)

* fix: Remove tag entity menu from LU Toolbar

* remove en-US changes

* minor

* remove test

* fix: Refactor LUIS, QNA and Speech to use a single base component (#6923)

* Refactor LUIS, QNA and Speech to use a single base component

* fix height, sort order

* fix typing

* use shared colors

* refactor handoff instructions to share some text

* Update BotProjectsSettingsTabView.tsx

* refactor qna maker provisioning into a background process

* fix linter quible:w

* remove dep

* rename setVisible to onToggleVisibility

* Use shared color

* add conditional around missing hostname

* add conditional around missing hostname

* remove duplicate import

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>

* fix: UI bugs in provisioning (#7130)

* Make firendly link and add blank targer for broken azure link

* Fix scrollilng in publish modal

* Fixed comment box in publish confirm modal

* configure resource ui fix #6886

* import existing resources fit and finish #6889

* #7132: standardizing qna plural/singular resource strings

* #7133: disabling Luis and Qna exit modal functionality when resources are being created

* #7126: remove semi colen

* Updating localized text

* Fixing inline conditionals

* Updating unit test

* Initial changes from UI meeting

* Removing unused imports

* Adding back changes reverted during merge

* fix: catch dotnet errors during creation and offer download link (#7062)

* catch dotnet error

* Handle error for missing dotnet 3.1 during creation

* remove unnecessary wrapper funct

* update to syntax of component, small cleanup items from PR

* remove old NodeModal (replaced by InstallDepMOdal)

Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Geoff Cox (Microsoft) <[email protected]>

* feat: add filter to begin dialog selection (#7193)

* make custom system topic icon more versitile

ex. it will now inherit font color from parent

* convert select dialog to a contextual menu

in preparation for adding filtering function

* correctly assess whether to show text field

* add filter to dialog list when more than 10

* align system topic centered vertically

* fix issue escaping search box with down arrow

* overflow dialog menu when more than 10 items

Co-authored-by: Chris Whitten <[email protected]>

* fix publish targets state update (#7207)

* fix: Recent bots sort and update styles (#7140)

* update styles

* recent bots list

* lint

* no hover color for table header

Co-authored-by: Ben Yackley <[email protected]>

* fix: LG goto definition lost some cross files definitions (#7203)

* fix lg goto missing

* remove log

Co-authored-by: Chris Whitten <[email protected]>

* fix: sort multi-skill bots in list by first opening of root (without skills) (#7185)

* add isRootBot argument to fetching projects

* Update Home.tsx

* Update project.test.ts

* Update en-US.json

* fix: reset dialog filter upon dismissing menu (#7222)

Co-authored-by: Soroush <[email protected]>

* Fix conditional rendering of runtime dropdown (#7228)

Co-authored-by: Chris Whitten <[email protected]>

* fix: minor UI bugs in provisioning and node modal (#7233)

* Fix conditional rendering of runtime dropdown

* #6890 add resource fit and finish

* Making design requested changes to provisioning form dialog

* Fixing styling in add resource modal

* Add need to restart in node modal

* Revert changes

* Updating template sort order list in anticipation of new package names

* - Adjusting padding to fit multipleof 4
- taking dashes out of formatMessage

* removing new line

* fix: add telemetry events for new qna, luis and speech flows (#7227)

* add telemetry events for new qna, luis and speech flows. add additional telemetry to get started.

* add additional telemetry calls to the provisioning process

* remove some unused imports

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* Update extensions/azurePublish/src/components/azureProvisionDialog.tsx

Co-authored-by: TJ Durnford <[email protected]>

* change name of event to ProvisionAddResourcesNavigate

* rename event

Co-authored-by: TJ Durnford <[email protected]>

* fix ux issues (#7242)

Co-authored-by: Soroush <[email protected]>

* fix: update UI in bot settings page to final R13 design spec (#7065)

* tab name changes

* Updating to latest UI: first/overview tab in settings page

* Adding (root) suffix to root bot in settings view nav pane

* removing old table view of bot settings page

* Fixing Luis and Qna styling

* Fixing fluentUI imports and skills settings page UI

* Fixing UI for appID component in settings page

* Fixing connections tab UI

* Fixing skills tab UI

* language tab UI fixes

* Adding teams callout when teams is enabled

* removing unused variables and deps

* Changing text with links to use formatMessage.rich

* Fix column width

* Fixing luis provision modal read me link

* Updating locale file

* Remove unused import

* Update unit test to reflect new tab text

* Add links to new hyperlinks in settings page

* Adding learnmore links to new qna/luis modal implementation

* Adding keys to rich format message

* updating locale file

* Making minor changes requested in PR

Co-authored-by: Soroush <[email protected]>

* fix: Sorted options on Azure & Luis region dropdowns (#7244)

* Sorted options on Azure regions & luis dropdowns

* Fixing the error from unit tests

Co-authored-by: Soroush <[email protected]>

* fix: widen column in feed modal (#7239)

Co-authored-by: Andy Brown <[email protected]>

* fix: introduce ability to pick a free or paid tier of QnA (#7184)

* Refactor LUIS, QNA and Speech to use a single base component

* fix height, sort order

* fix typing

* use shared colors

* refactor handoff instructions to share some text

* Update BotProjectsSettingsTabView.tsx

* refactor qna maker provisioning into a background process

* fix linter quible:w

* remove dep

* rename setVisible to onToggleVisibility

* Use shared color

* add conditional around missing hostname

* add conditional around missing hostname

* remove duplicate import

* add ability to specify pricing tiers when provisioing a qna

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* Do not specify the luis endpoint key as a parameter to the runtime if no vlaue is present (#7240)

(leaving this paramter blank causes issues on windows)

* disable telemetry calls in the provision dialog while we investigate why telemetryclient is null (#7256)

Co-authored-by: Geoff Cox (Microsoft) <[email protected]>

* prefer the botName field instead of the name field when managing connections (#7262)

* fix: Empty Webchat inspector text and Disabling items in PVA context (#7241)

* show floating notifications over eveything (#7269)

Co-authored-by: Soroush <[email protected]>

* Region for Microsoft Bot Channels Registration is now global (#7270)

Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: adjust package manager feeds (#7243)

* Fix #7092: set default page size to 100 items

* Fixes #6854: merge community feeds into main feed, sort by downloads

* Fixes #7043: include any component tagged msbot-component

* improve error handling

* restore different checks for declarative only vs code driven components

* refactor to use includes instead of indexOf

Co-authored-by: Ben Yackley <[email protected]>

* fix: remodel About page (#7191)

* remodel About page

* add SHA to version

* fixes from suggestion

* get info from Electron and use it for Release field

Co-authored-by: Chris Whitten <[email protected]>

* fix: Remove placeholder before saving import config (#7265)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix issues with existing name stopping is new

* Remove placeholders before saving import config

* Inline remove placeholders

* fix: error dialog should allow scrolling of error text (#7251)

* Issue-6727: update error popup consoleStyle to scroll, change background.

* Issue-6727: refactor format error title

* Issue-6727: use formatMessage, lighten up gray

Co-authored-by: natalgar <[email protected]>
Co-authored-by: pavolum <[email protected]>
Co-authored-by: Geoff Cox (Microsoft) <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: replace PVA with Power Virtual Agents (#7279)

* replace PVA with Power Virtual Agents

* update english locale

Co-authored-by: Ben Yackley <[email protected]>

* fix: Empty setting for recognizer configuration when switch bot (#7302)

* fix: Empty setting for recognizer configuration when switch bot

* update the naming

* fix: delete wrong trigger when delete skill and its trigger (#7305)

* fix repeated luIntent in add remote skill

* enus

* unblock diapatch modal is non

* add await in delete trigger

* fix trigger deleted, if no relevant trigger found

* fix: expose provisionComposer script in easily downloadable way (#7180)

* add an adminHandoff folder and new instructions

* rename folder

* add easily downloadable scripts

* direct download link for scripts

* direct download link for scripts

* correct numbers on steps

* fix: use correct worker script in electron development (#7274)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* fix: use router-based links to project manager (#7271)

* use router-based links to project manager

* fix unit test

* Update AdapterSettings.test.tsx

Co-authored-by: Andy Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>

* Fix for settings being overriden (#7294)

Co-authored-by: Andy Brown <[email protected]>

* fix: Change carousal to carousel in LG Editor (#7318)

* fix: Change carousal to carousel in LG Editor

* l10n

* Moved Web Chat log item keys outside of component (#7319)

* fix: Data loss when cutting / copying & pasting response (#7296)

Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: export bot include v2 project structure files (#7248)

* export bot include new structure files

* include js runtime type bot files

* clean up

Co-authored-by: Chris Whitten <[email protected]>

* fix: refactor way we get COMPOSER_VERSION (#7326)

Co-authored-by: Chris Whitten <[email protected]>

* fixes 6799 (#7320)

Co-authored-by: Soroush <[email protected]>

* fix: Update provisioning model based on feedback (#7328)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix issues with existing name stopping is new

* Remove placeholders before saving import config

* Inline remove placeholders

* Updated provisioning UI based on change requests

* Add formatMessages

* formatMessage

* Reverting cancel button spacing per design

* Updated link location to SDK release, adds link to documenation on the About page (#7339)

Co-authored-by: Soroush <[email protected]>

* Bumps to 2.0.0 (#7350)

* fix: fix LuEditor block entity in skill dialog (#7201)

* update orchestrator

* remove entity check in add remote skill

* update lueditor lufeature

* revert the changes about luFeatures

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: allower callers displayed like available to edit (#7361)

* callers ux refactor

* auto focus

* lint

Co-authored-by: Lu Han <[email protected]>

* fix: Orchestrator report entity error (#7362)

* Orchestrator report entity error

* set to warning

Co-authored-by: Dong Lei <[email protected]>

* fix: re-enable telemetry events in azure provision dialog (#7332)

* make shell available to all plugin types

* re-enable telemetry events in azure publish dialog

Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: allow scroll on editor parent (#7357)

Co-authored-by: Andy Brown <[email protected]>

* fix: adjust get started tasks (#7344)

* Add 2 new tasks to get started (view readme, complete profile) and make edit LU, edit LG recommended steps

* add formatMessage

Co-authored-by: Andy Brown <[email protected]>

* fix: better display and logic around handling remote skills (#7330)

* handle remote bots better in tree and package manager

* start plugin at root bot

* check remoteness and disable sidebar icons if needed

* Update pageLinks.ts

* fix: Resolve PVA check logic missing (#7369)

* Resolve minor logic error

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Hide item

Signed-off-by: Srinaath Ravichandran <[email protected]>

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: dummy skill  (#7306)

* Fix dummy skill issue

* dummy skill

* revert

Co-authored-by: Long Alan <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: share name field between two create kb modal (#7304)

* share knowledge base name between modal and update text

* update text

* fix UT

* handle comments

* update interface

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: Add hostname default to import publishing profile (#7370)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix issues with existing name stopping is new

* Remove placeholders before saving import config

* Inline remove placeholders

* Updated provisioning UI based on change requests

* Add formatMessages

* formatMessage

* Reverting cancel button spacing per design

* Added hostname default

Co-authored-by: Soroush <[email protected]>

* feat: Add RecognizerChanged telemetry event (#7374)

* feat: Add RecognizerChanged telemetry event

* Update RecognizerField.tsx

Co-authored-by: Soroush <[email protected]>

* Updating templates.json to remove old display name mappings and match desired ordering. (#7375)

* clean recent projects (#7379)

Co-authored-by: Soroush <[email protected]>

* feat: Move Webchat Panel inside the Page section to prevent overlap over Debug Pane (#7258)

* correct logic for validating connection modal (#7384)

Co-authored-by: Chris Whitten <[email protected]>

* Remove LUIS from trigger dialog (#7386)

Co-authored-by: Chris Whitten <[email protected]>

* fix: Update copy of provision step title (#7377)

* fix: 6566 Update provision handoff copy

* fix: 6851

* Moved sign-in to config page

* Fixed broken QnA image svg

* fix issues with existing name stopping is new

* Remove placeholders before saving import config

* Inline remove placeholders

* Updated provisioning UI based on change requests

* Add formatMessages

* formatMessage

* Reverting cancel button spacing per design

* Added hostname default

* Updated step title string

* fix plugin page router (#7400)

* fix: various template bugs (#7378)

* Fix qna template name and prepare sort order for new package names

* Fixing default proj name logic and minor cleaning up

* Partially fix initial template selection bug

* Add timeout to fully fix initial template selection UI bug

* fix qna read me bug

* fix backup sort order

* Adding new templates to sort template unit test

* Fix creation unit test to account for new template name

* Adjusting unit test to check for default name derived by display name vs template package name

* Catch common QNA maker error about limits (#7368)

Co-authored-by: Chris Whitten <[email protected]>

* fix #7403: include packages based on specific tags, rather than the msbot-component catchall (#7407)

* Fixed luis provisioning to return location (#7431)

Co-authored-by: Chris Whitten <[email protected]>

* fix: Add a warning about the luis authoring key 1000/month limit  (#7430)

* Add a warning about the luis authoring key 1000/month limit unless an endpoint key is set

* add space

Co-authored-by: Chris Whitten <[email protected]>

* fix: add skills flow refactored to desired R13 spec (#7432)

* Adding new creationflow status and setting of that status in new skills flow

* Adding new modal for 'add skills' flow per spect in #7115

* Changing dialog type to have proper size

Co-authored-by: Ben Brown <[email protected]>

* fix: UI fixes in getting started rocket panel  (#7418)

* Various style changes to resolve issue #6993

* fix style tagging

* Styling changes after review with Em

* fixing default naming

* fixing unit test to account for new default naming

* creating named css variable for rocket panel description styling

* Adding learn more links

* Cleaning up logic for default name generation

Co-authored-by: TJ Durnford <[email protected]>

* fix: bump bf-dialog to 4.13.1 (#7419)

* bump bf-dialog to 4.13.1

* update yarn locks

Co-authored-by: Chris Whitten <[email protected]>

* fix: address minor UX issues for PVA bots (#7437)

* update getting started blade for pva bots

* disable form dialogs link for pva bots

* disable entity buttons for pva bots

Co-authored-by: Chris Whitten <[email protected]>

* fix: when add remote skill show manifest json parse error (#7396)

* skill manifest show json parse error

* update

* Update CreateSkillModal.tsx

* Update CreateSkillModal.tsx

* update message

Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: Improve LG response editor parsing for text/speech variations (#7434)

* fix

* added comments

* minor

* lint

* lint

Co-authored-by: Soroush <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>
Co-authored-by: Dong Lei <[email protected]>

* Add azure functions existing when hostname there (#7455)

* Removed ARM & graph tokens on signout (#7457)

* fix: address missing links in provisioning, updated tags in package manager (#7458)

* fix #7454: add msbot-middleware to list of tags used to identify a code-driven pacakge

* Fix #7449: update links in provision flow

Co-authored-by: Chris Whitten <[email protected]>

* Fixed the buttons position on the provisioning modal. (#7459)

* fix: detect bots vs skills in Most Recently Used list (#7463)

* clean up a bit of redundant code

* detect root vs skill in import and creation

* Update project.ts

* fix: Added copy from profile button and dialog (#7465)

* Added copy from profile button and dialog

* Updates and fixes

* PR updates

* PR fixes

* More PR improvements

* Fix of import

* fix: hide recognizers if they don't exist in the app schema (#7469)

Co-authored-by: TJ Durnford <[email protected]>

* chore: Temporarily removes save as button for R13 (#7482)

* chore: Temporarily disable save as button for R13

* disable e2e save as test

* Updates Diagnostic message when a Skill is referenced that is not in the current project (#7481)

Co-authored-by: Soroush <[email protected]>

* fixes notification container blocking interaction (#7429)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: Add Publish your bot to get started list (#7484)

* Add Publish your bot to get started list

* add learn more link

Co-authored-by: Andy Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: enable users to select any non multi bot template during add new skills flow (#7494)

* Fix wording and routing in addBotModal

* Adding isMultiBot boolean value to botTemplate type

* setting isMultiBotTemplate based on keyword value from template generator

* Adding logic to not show multibot template if you are in the add skills flow

* Adjusting unit test for new botTEmplate multibot variable

* Adding formatMessages

* Removing unneeded code in conditional

* fix: update gitignore, detect missing func runtime (#7488)

* Fix #7473: add generated/ to .gitignore

* fix #7471: detect missing func runtime

* fix error link text

Co-authored-by: Soroush <[email protected]>

* fix: Handle expiration on token used for tenants (#7504)

* Handle expiration on token used for tenants

* Clear expiration on shutdown

* Fix unit test and no account response

* feat: auto publish after creating new provision. (#7249)

* add trigger for publish

* provision & publish

* async problem

* lint

* sync file change

* async publish target

* navigate to publish page after returning publish result

* add url

* comments

* bugifx

* lint

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>

* fix: allow dashes for pva bot names (#7500)

* allow dashes for pva bot names

* update v2 invalid name char

Co-authored-by: Ben Yackley <[email protected]>

* fix: Check for webchat panel visibility (#7514)

Signed-off-by: Srinaath Ravichandran <[email protected]>

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* Fix #7525: correct luis link (#7527)

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* feat: add telemetry around new bot creation (#7508)

* Adding missing types

* Simplifying createModal telem

* convert formatMessage variable to function

* Adding creation cancelled telemetry

* telemetry for provision profile creation failure

* Adding telemetry for request template button

* Adding telem for publish success and failure

* Adding telem for publish start button click

* Adding telem for create provision prof started with newResoruceGroup variable

* Adding unary null operator

* - making PR changes
- updating localization file

* removing ununsed imports

* Making requested PR changes

Co-authored-by: Ben Yackley <[email protected]>

* fix: Added sign out confirmation and notify user on sign out (#7537)

* Removed Signout Link & added title on Add new publishing profile

* Add Notification capability to extensions

* Added notification styles

* Modified authClient api

* Removed deplicate code

* Refactored signout confirmation code

* Added aria role to the div

* User will be notified when signout fails

* Changed the confirmation message & used the exisiting notificationType

* Removed unused type

Co-authored-by: Soroush <[email protected]>

* fix: Display Orchestrator Selection Dialog for Local Skills (#7518)

* First pass to add OrchSkillDialog to local skills

* Rename Orchestractor to Orchestrator

* update package to 4.13.1

* Address PR comments

* Use string constants where possible

* Rename Orchestrator component

* Fix another typo

* Change link to bf-orchestrator

* Link and copy changes per PM and Design

* add unit test

Co-authored-by: Chris Whitten <[email protected]>

* fix: Updates AllowedCallers component to use unique keys for array items (#7540)

* fix: Updates AllowedCallers component to use unique keys for array items

* requested changes

* fix: correct the MS AppID text in the settings view (#7552)

* #7426 Fix appsettings copy

* localization updates

* fix: Add a QNA trigger to the dialog if it doesnt exist (#7534)

* Add a QNA trigger to the dialog if it doesnt exist -- when adding a KB in the KB page

* remove space

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* fix: split "Add new" button into its own section on Publish Targets page (#7512)

* Update PublishTargets.tsx

* streamline some logic and rename variables for clarity

* fix: Using Get LUIS or Get QnA Key option does not work in Composer  (#7553)

* Added auth flow MSA Accounts

* Code review changes

* Removed unnecessary code

* changed function from 'subscriptionSelector' to 'renderSubscriptionSelectionStep'

* Minor UI fixes

* Added new strings to en-US.json

* Formatted regions on LUIS/QnA resource creation dialog and  removed unused code.

Co-authored-by: pavolum <[email protected]>

* fix: Reopening of dialog due to hash (#7559)

* Fixed reopening of dialog due to hash

* Added back dependencies

* fix: Updated the cypress test for the integration test to succeed (#7575)

* Added auth flow MSA Accounts

* Code review changes

* Removed unnecessary code

* changed function from 'subscriptionSelector' to 'renderSubscriptionSelectionStep'

* Minor UI fixes

* Added new strings to en-US.json

* Formatted regions on LUIS/QnA resource creation dialog and  removed unused code.

* Updated cypress test

Co-authored-by: pavolum <[email protected]>

* fix: Update attachment template format to support PVA (#7564)

* fix: Update attachment template format to support PVA

* minor changes

* fix tests

* fix lint issue

Co-authored-by: Soroush <[email protected]>

* Revert "fix: Update attachment template format to support PVA (#7564)" (#7592)

This reverts commit ee715eb9ee336dce7aabbb2eed585f65ef4dfc3d.

* fix: Update attachment template format to support PVA (#7593)

minor changes
fix tests
lint

* fix: address a regression in the start bot button that caused final 'get started' bubb not to pop (#7567)

* fix #7560: address a regression in the start bot button

* change name of element

Co-authored-by: Soroush <[email protected]>

* fix: get tenant specific arm tokens for connections  (#7554)

* fix #7536: get tenant specific arm tokens

* combine auth into 1 function

* Attempt various ways to determine tenantId

* only set current resource if auth was successful

Co-authored-by: Andy Brown <[email protected]>

* fix: change copy on home page (#7604)

* edit strings to match suggestions

* Update Home.tsx

* Update Home.tsx

* fixes from review and e2e test

* l10n update

* Update provisionScript.zip with small syntax changes (#7563)

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: Adjust build command used for dotnet functions (#7557)

* Adjust build command used for dotnet functions

* pass in the appid/passwd to functions

* correct source of appid/password ford func deploy

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: fixes auth flow on web & electron version (#7623)

* Added auth flow MSA Accounts

* Code review changes

* Removed unnecessary code

* changed function from 'subscriptionSelector' to 'renderSubscriptionSelectionStep'

* Minor UI fixes

* Added new strings to en-US.json

* Formatted regions on LUIS/QnA resource creation dialog and  removed unused code.

* Updated cypress test

* Fixed the auth flow

* Tenant dropdown fix

Co-authored-by: pavolum <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: ui copy updates (#7599)

* copy updates

* Fixes for LUIS service copy

* adjustments for qna and speech copy

* qna maker copy changes

* copy changes

* casing

* address comments

* update locale

* fix test

* fix linter problem

* fix typescript

* fix: LG Editor Fit & Finish (#7636)

* fix: LG Editor Fit & Finish

* l10n

* fix: rephrase Open from ABS dialog (#7637)

* Update constants.ts

* replace text on choices

* remove Learn More for now

Co-authored-by: Soroush <[email protected]>

* fix: fit and finish across the app (#7598)

* Remove bot settings tool bar buttons

* Change settings header string and remove subtext

* Fix capitalization in settings tabs

* More space between pivot menu and content--should be consistent on each tab

* - Align content with pivot menu on left side
- Max width of inputs and text 580x in all tabs

* In all tables, headers should be semi-bold rather than bold

* nix tool tip on start command

* "Readme" instead of "Read Me"

* Ensure padding at bottom of page

* Tighten up horizontal spacing between columns in tables

* Replacing routerLink with Link for consistent styling

* Lower-case M in "Open manifest"

* - Action item sub-text too small
 - Tighten space between action items more
 (between bottom of sub-text and title of next item)

* Sub-titles should be semi-bold

* - Align table with “Publish” (move to the right)

* - "Publishing profile"--lowercase on profile

* - Super long add button (#7491)

* Table needs line under header

* Edit and delete buttons are too small, too high

* Align Target column content with header (move content right)

* Tighten spacing between edit and delete buttons

* "Learn more" move to top sentence after "...Microsoft Azure is required." and lowercase m

* - Lowercase m

* - Do we need description field? Can we remove it?
 - If so, maybe runtime type on right?
 - Inputs same width (370px?)
 - “Create new folder” under input

* Design iteration changes with Em

* add line under bot settings header

* Almost there" bubb is the wrong component, should be the blue one

* urther Tighten spacing between edit and delete buttons in publish view

* DefineConversation: Button should say “Create” instead of OK

* Move json switch to right side and make text smaller size

* Fix existing resource terminology in service provision modal

* #7578: increase width of publish profile dropdown

* Auto close get started pane when bot controller callout is visible

* fixing errors from merge

* Fix unit test

* removing unit test based on old link type

* updating locale strings

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* special case the fallback recognizer (#7634)

* chore: bump wait-on to v5.3.0 (#7648)

* fix: update left nav and top bar strings (#7609)

* start fixing header

* adjust header chrome and left nav order

* Update en-US.json

* fix typecheck errors

* fix unit tests

* Update en-US.json

* change Design to Create in e2e tests

* Update LuisDeploy.spec.ts

* update flaky check in visitPage

* add 'checked' flag to visitPage to minimize surface area

* fix punctuation

* fix e2e test!

* fix security issue

* Revert "fix security issue"

This reverts commit aa008c1f7857667b5e2f4cb3e0fcedbb8c1ad7af.

Co-authored-by: Chris Whitten <[email protected]>

* fix: rephrase Enable Orchestrator dialog (#7639)

* "Add a skill" page changes

* Update EnableOrchestrator.tsx

* Update skill.test.tsx

* Update package.json

* Update yarn.lock

* Update en-US.json

* fix: Update provision & publishing copy (#7583) (#7642)

* Issue-7583: update copy for provision and publishing

* en-us json files

* Issue-7583: fix formatDialogTitle

* Issue-7583: refactor formatDialogTitle

* Issue-7583: reformat azure provision dialog block

* PR fixes

* Fixed spacing

Co-authored-by: GeoffCoxMSFT <[email protected]>

* fix: Added cleanup logic to app on quit (#7645)

* Added cleanup logic to app on quit

* Added missing electron ipc event listener

Co-authored-by: TJ Durnford <[email protected]>
Co-authored-by: Soroush <[email protected]>

* feat: show tunneling information notification when starting bot with remote skills (#7611)

* add util to get device OS

* add port to local publish result

* add port to botEndpointsState

* show ngrok notification when starting a bot with remote skills

* only show the remote skills notification onces per session per bot

* update l10n file

* address feedback

* update locale file after merge

* fix failing test

Co-authored-by: Soroush <[email protected]>

* feat: detect "old" bots and migrate them to new runtime (#6526)

* introduce a migration system for updating legacy projcts to the 2.0 runtime

* updated

* ensure root dialog gets updated appropriately

* adjust method for updating root dialog

* remove comments

* fire migration warning when loading bot

* use original project name for new project

* close modal on submit

* address feedback

* add confirm modal

* Rename azureFunctionsPublish publish targets to azurePublish (new shared id)

* Delete a.en-us.lu

* Delete a.en-us.qna

* Delete b.en-us.lu

* Delete b.en-us.qna

* Delete bot1.en-us.lu

* Delete bot1.en-us.qna

* do not require pva to migrate

* Add missing parameters

* clarify types

* Fix tests
reduce redundant code

* Fixes #6844: migrate appinsights key to new location

* fix migrate lint issue

* plumb through the yeomanOptions parameter, required to set the location of the settings folder when migrating

* feat: new validation pipeline - schema existence validation (#7001)

* add placeholder for schema validator

* add schema validator pipeline with mocked fn

* add schema visitor

* display diagnostics data in debug panel

* revert sdk.ts

* decrease schema diagnostic severity to 'Warning'

* optmize path join logic

* impl a unified walker covers SwitchCondition

* fix lint error: use BaseSchema

* feat: disable actions without schema

* wrap in useEffect

* optimization: avoid frequent recoil submission

* optimization: aggregate paths rather than updatedDialog to reduce time complexity

* chore: comments & var name

* lint

* add comments

* defense undefined skip-level 'actions'

* defense potential exceptions

* get sdk.schema content correctly

* fix lint

* fix folder name case problem

* Do not specify the luis endpoint key as a parameter to the runtime if no vlaue is present (#7240)

(leaving this paramter blank causes issues on windows)

* disable telemetry calls in the provision dialog while we investigate why telemetryclient is null (#7256)

Co-authored-by: Geoff Cox (Microsoft) <[email protected]>

* prefer the botName field instead of the name field when managing connections (#7262)

* fix: Empty Webchat inspector text and Disabling items in PVA context (#7241)

* show floating notifications over eveything (#7269)

Co-authored-by: Soroush <[email protected]>

* Region for Microsoft Bot Channels Registration is now global (#7270)

Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: adjust package manager feeds (#7243)

* Fix #7092: set default page size to 100 items

* Fixes #6854: merge community feeds into main feed, sort by downloads

* Fixes #7043: include any component tagged msbot-component

* improve error handling

* restore different checks for declarative only vs code driven components

* refactor to use includes instead of indexOf

Co-authored-by: Ben Yackley <[email protected]>

* fix: remodel About page (#7191)

* remodel About page

* add SHA to version

* fixes from suggestion

* get info from Electron and use it for Release field

Co-authored-by: Chris Whitten <[email protected]>

Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Geoff Cox (Microsoft) <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Vamsi Modem <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>

* fix mapping of schema files during migration

* fix lint

* fix default name of migrated project

* run schema merge as part of the migration

* do we need to migrate teh schema? or not?

* do not migrate schema

* - Adding field for runtime language
- Fixing field population for runtime type

* Passing runtime lang and type down to yeoman calls

* Fixing adaptive runtime name

* Hardcoding runtime version and fixing settings page generation on migrate

* Fixing check for 'inBotMigration' to be based on CreationStatusState > path substring

* Fix default naming

* revert preload.js

* Resolving PR comments and fixing errors from merge

* Change schema diagnostics severity to 'Error'

* Fetch @latest version from npm before migration

Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: leilzh <[email protected]>
Co-authored-by: zhixzhan <[email protected]>
Co-authored-by: zeye <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Geoff Cox (Microsoft) <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Vamsi Modem <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Patrick Volum <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: data race writing on setting file (#7475)

* fix data race writing on setting file

* remove isSkill in skillConfiguration

* remove default allowedCallers value

Co-authored-by: TJ Durnford <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Soroush <[email protected]>

* remove appsettings file from default gitignore (#7679)

* fix: restore line beneath page header (#7676)

* Update Page.tsx

* Update BotProjectSettings.tsx

* chore: Rebase main with 1.4.1 release (#7678)

* exclude items from the package manager list that do not have names or versions -- such as locally implemented custom actions (#7683)

* Add components field to migrated settings. Guard against missing field during component merge (#7674)

Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: (Un) Installing a package now stops the currently running bot beforehand (#7689)

* Feed string, not int, to Switch per schema (#7707)

Co-authored-by: Chris Whitten <[email protected]>

* fix: generate manifest file when creating new publish target (#7666)

* generate manifest file when creating new profile

* test

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>

* fix: converting bot with custom actions (ui update) (#7672)

* Hides the PVA publish profile from the profle creation dropdown (#7725)

* fix: set func-related settings during build step (#7723)

* set func-related settings during build step

* slight refactor to reduce duplicated code

* add type

* add typings in runtime plugin definition

* set type of port to number

* fix types, remove _ on variables that are used

* feat: Add preparatory work before connecting remote skill (#7519)

* draft dialog

* ux css

* css

* jump to create profile & set microsoftAppId to publish target

* comments & lint

* refactor dialog wrapper

* fix publish types missing in provision dialog (#7697)

* test case

* refactor

* fix type define

* fix title and json parse

* fix app Id not sync when create new profile

* Adds AppID and Password sections

* test case fixed

* Adds AppID/Password

* Update copy

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: VanyLaw <[email protected]>
Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: Revert changes to adaptive card templates to support PVA (#7808)

* fix: Revert changes to adaptive card templates to support PVA

* requested changes

* fetch publish types for each project

* fix: Throttle restart conversation (#7824)

* Throttle restart

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Remove await

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Updated throttle time

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Unit test update

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Enable only on connected

Signed-off-by: Srinaath Ravichandran <[email protected]>

* removed if

Signed-off-by: Srinaath Ravichandran <[email protected]>

Co-authored-by: Srinaath Ravichandran <[email protected]>

* chore: automated localization updates (#7759)

* Localized resource files from OneLocBuild

* fix typos and stray apostrophes

* fix apostrophes

* typo fix

* Update en-US.json

* fix more apostrophes

* revert reversions

Co-authored-by: Composer Localization <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* Deploy exception returns real error (#7837)

* fix: Fix missing data collection settings on server (#7814)

* fix: Fix missing data collection settings on server

* fix tests

* fix test again

* minor change

Co-authored-by: Chris Whitten <[email protected]>

* fix: add inner scrollbar to selection area (#7782)

* scroll

* css

* test

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* Migration: adjust target blob transcript field names (#7848)

* fix: Always use intermediate lg template for text and speak modalities (#7842)

* allow click outside blocking modals (#7727)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>

* fix: Remove depth and transparentBorders from FieldProps, remove ObjectArrayField, and refactor OpenObjectField (#6633)

* Remove depth and transparentBorder

* Update OneOfField dropdown styles

* Remove ObjectArrayField

* fix tests

* Refactor OpenObjectField

* updated test

* minor

* fix test

* minor

Co-authored-by: Ben Yackley <[email protected]>

* fix: Teaching bubble missing content (#7764)

* fix: Teaching bubble missing content

* Fix item ref

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: Reset allowed callers cache when switching between project settings (#7874)

* fix: Reset allowed callers cache when switching between project settings

* comment

Co-authored-by: Soroush <[email protected]>

* fix: breadcrumb & show code button collision (#7720)

* fix breadcrumb & show code button collision

* use flex

* update styles

* observe button width, auto layout ... breadcrumb

* ignore typescript ResizeObserver declarations missing

* fix type in test

* useResizeObserver

* fix mock

Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Soroush <[email protected]>

* Update home feed for Build 2021 (#7912)

* fix (#7440)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: TJ Durnford <[email protected]>

* fix: Custom function declaration (#7775)

* Custom function declaration

Signed-off-by: Srinaath Ravichandran <[email protected]>

* Typo in custom functions

Signed-off-by: Srinaath Ravichandran <[email protected]>

Co-authored-by: Srinaath Ravichandran <[email protected]>

* add new caller when caller still have empty list, just focus on the empty one (#7767)

Co-authored-by: Lu Han <[email protected]>

* Resolve dns-packet to 1.3.4 to fix CVE-2021-23386 (#7942)

Co-authored-by: Soroush <[email protected]>

* fix: invalid profile case (#7784)

* invalid profile case

* comments

* remove unused component

* refactor publish profile wrapper dialog

* toekn page jump logic err

Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Soroush <[email protected]>

* update recognizer

* update recognizer

* polish recognizer UI

* polish recognizer UI

* polish

* polish

* fix test

* fix test

* fix e2e test

* fix e2e test

* fix: Need to wait for zip deployment to be done processing (#7858)

* Handled 202

* Added error case for processing

* Improvements

Co-authored-by: Soroush <[email protected]>

* Update readme screenshot (#7918)

Co-authored-by: Soroush <[email protected]>

* fix comments

* fix comments

* fix lint and ut

* fix lint and ut

* upgrade json-ptr (#7988)

* fix (#7979)

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* feat: Add expand to larger editor to LU Editor (#7976)

* pop expand lu editor

* fix labeling menu zIndex in poped out editor

* pr comments

Co-authored-by: Soroush <[email protected]>

* fix: LUIS build fails with reason "undefined" with more than 10 phraselist entities (#7399)

* fix luis build error undeifined

* update to dev version

* update bf-lu version

Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Tien Suwandy <[email protected]>

* chore(deps): bump ws from 7.4.4 to 7.4.6 in /extensions/localPublish (#7962)

Bumps [ws](https://github.com/websockets/ws) from 7.4.4 to 7.4.6.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](https://github.com/websockets/ws/compare/7.4.4...7.4.6)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: Support vertical bar character ("|") in MicrosoftPassword (#7938)

* fix command with special char

* change quo mark

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Dong Lei <[email protected]>

* ci: increase unit test timeout to 45 minutes (#7836)

Co-authored-by: Soroush <[email protected]>

* fix: rewrite l10n:check to catch more errors and fix some l10n issues (#7857)

* Update l10nCheck.js

* revise l10n:check script and fix more l10n issues

* add Unicode flag to regex and fix more quotes

* update l10n strings

* apply fix from comment

* when add language luFile, update luIntent fileId (#8009)

* chore: Upgrade bf-lu to latest dev version (#7999)

* upgrade bf-lu

* update yarn lock

* chore: upgrade package [email protected] (#8010)

Co-authored-by: Soroush <[email protected]>

* fix: trigger creation LUIS entity labelling (#8011)

* fix: trigger creation LUIS entity labelling

* move useState to parent component

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Dong Lei <[email protected]>

* Add Orchestrator Logging (#7038)

* chore: bump ws from 7.4.4 to 7.4.6 and browserslist from 4.16.3 to 4.16.6. in /extensions/azurePublish (#8012)

* upgrade ws package

* upgrade browserslist

Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Andy Brown <[email protected]>

* fix: add keys to tags and split license notice by lines (#7389)

* add keys and split license notice by lines

* Update About.tsx

* generate unique IDs for repeated P tags

Co-authored-by: TJ Durnford <[email protected]>
Co-authored-by: Soroush <[email protected]>

* feat: Added UX to link log items and WC activities (#8006)

* Added UX to link log items and WC activities

* Updated selected visual state of WC activities

* Added some tests

* Factored out WebChat hooks into separate headless component

Co-authored-by: Andy Brown <[email protected]>

* fix comments

* fix comments

* fix: remove old creation code (#8004)

* Removing old components

* marking unit tests for revision

* Removing old creation string

* Updating CreateBot component

* Moving creation files outside of v2 dir

* removing V2 from creation class names

* update migrate and createProjcet api endpoints

* Updating fetchtemplates endpoint and references

* misc v2 verbage and variable name clean up

* update createbot dispatchers

* update createNewBotFromTemplate

* minor fix in navigation in creation flow

* Updating creation flow unit test location

* updating lcoale file and removing unused variable

* Fix Unit test mocking

* Remove old unit test

* reverting en-us to resolve conflict with main

* Rename article to be less confusing (#8030)

Per @cwhitten 

> The recommended OS versions are unchanged as of 1.3. We could modify this document to say something like 1.3 and above to be more correct.

* upgrade css-what and css-select (#8041)

* feat: Add dynamic schema for build forms in form dialogs (#8046)

* form dialog generators

* revert

* strings

* test

* chore: Update to the latest generator (#7664)

* Update to .template and new generate call.

* Update latest generator

Co-authored-by: Chris McConnell <chrimc>

* Remove debugger

* Make .template optional in $ref.

* Update to the npm version of bf-generate-library.

* #chore update to latest bf-generate-library

* Update to latest generator which fixes issues in
merge.

* Fix a bug in property grouping for forms.
Changed it so the first of:
* $designer.propertyGroups
* trigger.property
* action.expectedProperties
Is used for the property.
Enable multi-parenting and if not turned on
ensure the trigger does not disappear.

* strings

* yarn update

* lint issues

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris McConnell <chrimc>
Co-authored-by: Chris McConnell <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: GeoffCoxMSFT <[email protected]>

* fix: Monaco editor loader CDN js path (#7968)

* reset monaco loader CDN js path

* upgrade monaco-editor from 0.19.0 to 0.21.2

Co-authored-by: Andy Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>

* fix: incorrect expression validation for dialog (#7722)

* fix: incorrect expression validation for dialog

* fix conner case

* update the comments

* add some comment

* fix lint

Co-authored-by: Dong Lei <[email protected]>

* feat: Add T&C accept for teams (#7381)

* Add T&C accept for teams

* remove mystery dep

* remove console

* Update formatMessage to use formatMessage.rich for complex content

Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>

* fix: resolving getting started teaching callout not rendering (#8005)

* -Adding getstarted teach bubble to app state
-creating a dispatcher for it

* setting state value to show teaching bubble as opposed to hash routing

* adding onclick state changing to manage state for teaching button dismisal

* Refining naming for distinct teaching bubbles

* polish design

* polish design

* update url

* default selected

* fix comment

* fix e2e test

Co-authored-by: pavolum <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Chris Whitten <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Srinaath Ravichandran <[email protected]>
Co-authored-by: Ben Brown <[email protected]>
Co-authored-by: Shuai Wang <[email protected]>
Co-authored-by: Ben Yackley <[email protected]>
Co-authored-by: Tony Anziano <[email protected]>
Co-authored-by: Andy Brown <[email protected]>
Co-authored-by: Soroush <[email protected]>
Co-authored-by: Zhixiang Zhan <[email protected]>
Co-authored-by: Geoff Cox (Microsoft) <[email protected]>
Co-authored-by: Lu Han <[email protected]>
Co-authored-by: Qi Kang <[email protected]>
Co-authored-by: Long Jun <[email protected]>
Co-authored-by: Long Alan <[email protected]>
Co-authored-by: taicchoumsft <[email protected]…

Composer/packages/adaptive-form/src/components/fields/RecognizerField/RecognizerField.tsx

@@ -1,59 +1,160 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 /** @jsx jsx */
-import { jsx } from '@emotion/core';
-import React, { useMemo } from 'react';
+import { jsx, css } from '@emotion/core';
+import React, { useCallback, useMemo, useState, useEffect } from 'react';
 import { FieldProps, useShellApi, useRecognizerConfig } from '@bfc/extension-client';
 import { MicrosoftIRecognizer } from '@bfc/shared';
-import { Dropdown, ResponsiveMode, IDropdownOption } from 'office-ui-fabric-react/lib/Dropdown';
 import formatMessage from 'format-message';
+import { Dialog, DialogFooter, DialogType } from 'office-ui-fabric-react/lib/Dialog';
+import { ScrollablePane, ScrollbarVisibility } from 'office-ui-fabric-react/lib/ScrollablePane';
+import { Link } from 'office-ui-fabric-react/lib/Link';
+import { CheckboxVisibility, DetailsList, SelectionMode, Selection } from 'office-ui-fabric-react/lib/DetailsList';
+import { PrimaryButton, DefaultButton } from 'office-ui-fabric-react/lib/Button';
 
 import { FieldLabel } from '../../FieldLabel';
 
 import { useMigrationEffect } from './useMigrationEffect';
-import { mapDropdownOptionToRecognizerSchema } from './mappers';
-import { getDropdownOptions } from './getDropdownOptions';
+import { mapListItemsToRecognizerSchema } from './mappers';
+import { getDetailsListItems } from './getDetailsListItems';
 
+const recognizerStyle = css`
+  display: flex;
+  justify-content: space-between;
+  margin: 5px 0px 10px 0px;
+`;
+const AzureBlue = '#0078D4';
+const recognizerContainer = css`
+  position: relative;
+  height: 500px;
+  border-top: 1px solid #f3f2f1;
+  border-bottom: 1px solid #f3f2f1;
+`;
+const learnRecognizerUrl = 'https://docs.microsoft.com/en-us/composer/concept-dialog?tabs=v2x#recognizer';
+
+export type RecognizerListItem = {
+  key: string;
+  text: string;
+  description: string;
+};
 export const RecognizerField: React.FC<FieldProps<MicrosoftIRecognizer>> = (props) => {
   const { value, id, label, description, uiOptions, required, onChange } = props;
   const { shellApi, ...shellData } = useShellApi();
   const { telemetryClient } = shellApi;
+  const [showDialog, setShowDialog] = useState(false);
+  const [selectedRecognizer, setSelectedRecognizer] = useState<RecognizerListItem>();
 
   useMigrationEffect(value, onChange);
   const { recognizers: recognizerConfigs, currentRecognizer } = useRecognizerConfig();
-  const dropdownOptions = useMemo(() => getDropdownOptions(recognizerConfigs, shellData, shellApi), [
+  const detailsListItems = useMemo(() => getDetailsListItems(recognizerConfigs, shellData, shellApi), [
     recognizerConfigs,
   ]);
 
   const RecognizerEditor = currentRecognizer?.recognizerEditor;
   const widget = RecognizerEditor ? <RecognizerEditor {...props} /> : null;
 
-  const submit = (_, option?: IDropdownOption): void => {
-    if (!option) return;
+  const selection = useMemo(() => {
+    return new Selection({
+      onSelectionChanged: () => {
+        const selectedItems = selection.getSelection() as RecognizerListItem[];
+        if (selectedItems?.length > 0) {
+          setSelectedRecognizer(selectedItems[0]);
+        }
+      },
+    });
+  }, [setSelectedRecognizer]);
 
-    const recognizerDefinition = mapDropdownOptionToRecognizerSchema(option, recognizerConfigs);
+  const submit = useCallback((): void => {
+    if (!selectedRecognizer) return;
+
+    const recognizerDefinition = mapListItemsToRecognizerSchema(selectedRecognizer, recognizerConfigs);
 
     const seedNewRecognizer = recognizerDefinition?.seedNewRecognizer;
     const recognizerInstance =
       typeof seedNewRecognizer === 'function'
         ? seedNewRecognizer(shellData, shellApi)
-        : { $kind: option.key as string, intents: [] }; // fallback to default Recognizer instance;
+        : { $kind: selectedRecognizer.key as string, intents: [] }; // fallback to default Recognizer instance;
     onChange(recognizerInstance);
-    telemetryClient?.track('RecognizerChanged', { recognizer: option.key as string });
-  };
+    telemetryClient?.track('RecognizerChanged', { recognizer: selectedRecognizer.key as string });
+  }, [selectedRecognizer, recognizerConfigs, shellData, shellApi]);
+
+  useEffect(() => {
+    if (selection && currentRecognizer) {
+      selection.setItems(detailsListItems, false);
+      selection.setKeySelected(currentRecognizer.id, true, false);
+    }
+  }, [detailsListItems]);
 
   return (
     <React.Fragment>
       <FieldLabel description={description} helpLink={uiOptions?.helpLink} id={id} label={label} required={required} />
-      <Dropdown
-        ariaLabel={formatMessage('Recognizer Type')}
-        data-testid="recognizerTypeDropdown"
-        label={formatMessage('Recognizer Type')}
-        options={dropdownOptions}
-        responsiveMode={ResponsiveMode.large}
-        selectedKey={currentRecognizer?.id}
-        onChange={submit}
-      />
+      <div style={{ fontWeight: 600 }}>{formatMessage('Recognizer/Dispatch type')}</div>
+      <div css={recognizerStyle} data-testid="recognizerTypeList">
+        <span>
+          {typeof currentRecognizer?.displayName === 'function'
+            ? currentRecognizer?.displayName({})
+            : currentRecognizer?.displayName}
+        </span>
+        <span data-testid="openRecognizerDialog" style={{ color: AzureBlue }} onClick={() => setShowDialog(true)}>
+          {formatMessage('Change')}
+        </span>
+      </div>
+      <Dialog
+        dialogContentProps={{
+          type: DialogType.normal,
+          title: formatMessage('Choose a recognizer type'),
+        }}
+        hidden={!showDialog}
+        minWidth={450}
+        modalProps={{
+          isBlocking: true,
+          isClickableOutsideFocusTrap: true,
+        }}
+        onDismiss={() => setShowDialog(false)}
+      >
+        <div css={recognizerContainer}>
+          <ScrollablePane
+            scrollbarVisibility={ScrollbarVisibility.auto}
+            styles={{ root: { width: '100%', height: 'inherit', position: 'relative' } }}
+          >
+            <DetailsList
+              checkboxVisibility={CheckboxVisibility.always}
+              columns={[
+                {
+                  key: 'recognizer',
+                  name: 'Recognizer type',
+                  minWidth: 70,
+                  onRender: (item) => {
+                    return (
+                      <div>
+                        <div style={{ fontWeight: 'bold' }}>{item.text}</div>
+                        <div style={{ whiteSpace: 'normal' }}>{item.description}</div>
+                      </div>
+                    );
+                  },
+                },
+              ]}
+              isHeaderVisible={false}
+              items={detailsListItems}
+              selection={selection}
+              selectionMode={SelectionMode.single}
+            />
+          </ScrollablePane>
+        </div>
+        <DialogFooter>
+          <Link href={learnRecognizerUrl} styles={{ root: { fontSize: '12px', float: 'left' } }} target="_blank">
+            {formatMessage('Learn more about recognizers')}
+          </Link>
+          <DefaultButton text={formatMessage('Cancel')} onClick={() => setShowDialog(false)} />
+          <PrimaryButton
+            text={formatMessage('Done')}
+            onClick={() => {
+              setShowDialog(false);
+              submit();
+            }}
+          />
+        </DialogFooter>
+      </Dialog>
       {widget}
     </React.Fragment>
   );

Composer/packages/adaptive-form/src/components/fields/RecognizerField/getDropdownOptions.ts → Composer/packages/adaptive-form/src/components/fields/RecognizerField/getDetailsListItems.ts

@@ -5,7 +5,7 @@ import { RecognizerSchema, FallbackRecognizerKey, ShellApi, ShellData } from '@b
 import { checkForPVASchema } from '@bfc/shared';
 
 import { recognizerOrderMap } from './defaultRecognizerOrder';
-import { mapRecognizerSchemaToDropdownOption } from './mappers';
+import { mapRecognizerSchemaToListItems } from './mappers';
 
 const getRankScore = (r: RecognizerSchema, shellData: ShellData, shellApi: ShellApi) => {
   // Always put disabled recognizer behind. Handle 'disabled' before 'default'.
@@ -17,7 +17,7 @@ const getRankScore = (r: RecognizerSchema, shellData: ShellData, shellApi: Shell
   return recognizerOrderMap[r.id] ?? Number.MAX_VALUE - 1;
 };
 
-export const getDropdownOptions = (configs: RecognizerSchema[], shellData: ShellData, shellApi: ShellApi) => {
+export const getDetailsListItems = (configs: RecognizerSchema[], shellData: ShellData, shellApi: ShellApi) => {
   const isPVASchema = checkForPVASchema(shellData.schemas?.sdk);
   let recognizerConfigs: RecognizerSchema[] = configs;
   if (isPVASchema) {
@@ -31,5 +31,5 @@ export const getDropdownOptions = (configs: RecognizerSchema[], shellData: Shell
     .sort((r1, r2) => {
       return getRankScore(r1, shellData, shellApi) - getRankScore(r2, shellData, shellApi);
     })
-    .map(mapRecognizerSchemaToDropdownOption);
+    .map(mapRecognizerSchemaToListItems);
 };

Composer/packages/adaptive-form/src/components/fields/RecognizerField/mappers.ts

@@ -2,14 +2,16 @@
 // Licensed under the MIT License.
 
 import { RecognizerSchema } from '@bfc/extension-client';
-import { IDropdownOption } from 'office-ui-fabric-react/lib/Dropdown';
 
-export const mapDropdownOptionToRecognizerSchema = (option: IDropdownOption, recognizerConfigs: RecognizerSchema[]) => {
-  return recognizerConfigs.find((r) => r.id === option.key);
+import { RecognizerListItem } from './RecognizerField';
+
+export const mapListItemsToRecognizerSchema = (item: RecognizerListItem, recognizerConfigs: RecognizerSchema[]) => {
+  return recognizerConfigs.find((r) => r.id === item.key);
 };
 
-export const mapRecognizerSchemaToDropdownOption = (recognizerSchema: RecognizerSchema): IDropdownOption => {
-  const { id, displayName } = recognizerSchema;
+export const mapRecognizerSchemaToListItems = (recognizerSchema: RecognizerSchema) => {
+  const { id, displayName, description } = recognizerSchema;
   const recognizerName = typeof displayName === 'function' ? displayName({}) : displayName;
-  return { key: id, text: recognizerName || id };
+  const recognizerDescription = typeof description === 'function' ? description({}) : description;
+  return { key: id, text: recognizerName || id, description: recognizerDescription };
 };

Composer/packages/adaptive-form/src/components/fields/__tests__/RecognizerField.test.tsx

@@ -2,7 +2,7 @@
 // Licensed under the MIT License.
 
 import React from 'react';
-import { render, fireEvent, screen } from '@botframework-composer/test-utils';
+import { render, fireEvent, screen, getAllByText } from '@botframework-composer/test-utils';
 import { useRecognizerConfig, useShellApi } from '@bfc/extension-client';
 import assign from 'lodash/assign';
 
@@ -37,12 +37,14 @@ describe('<RecognizerField />', () => {
       {
         id: 'one',
         displayName: 'One Recognizer',
+        description: 'test1',
         isSelected: () => false,
         seedNewRecognizer: handleChange,
       },
       {
         id: 'two',
         displayName: 'Two Recognizer',
+        description: 'test2',
         isSelected: () => true,
         seedNewRecognizer: jest.fn(),
       },
@@ -51,12 +53,17 @@ describe('<RecognizerField />', () => {
       recognizers,
       currentRecognizer: recognizers[1],
     });
-    const { getByTestId } = renderSubject({ value: { $kind: 'two' } });
-    const dropdown = getByTestId('recognizerTypeDropdown');
-    expect(dropdown).toHaveTextContent('Two Recognizer');
-    fireEvent.click(dropdown);
+    const { getByText, getAllByText } = renderSubject({ value: { $kind: 'two' } });
+    // two recognizer already choosed
+    expect(getByText('Two Recognizer')).not.toBeNull();
+
+    // click change recognizer, pop up dialog
+    fireEvent.click(getByText('Change'));
+    expect(getByText('One Recognizer')).not.toBeNull();
+    expect(getAllByText('Two Recognizer').length).toBe(2);
 
     fireEvent.click(screen.getByText('One Recognizer'));
+    fireEvent.click(screen.getByText('Done'));
     expect(handleChange).toHaveBeenCalled();
   });
 });

Composer/packages/client/__tests__/plugins.test.ts

@@ -53,25 +53,25 @@ describe('mergePluginConfigs', () => {
     const config1 = {
       uiSchema: {
         [SDKKinds.RegexRecognizer]: {
-          recognizer: { displayName: 'recognizer1' },
+          recognizer: { displayName: 'recognizer1', description: 'recognizer1' },
         },
       },
     };
 
     const config2 = {
       uiSchema: {
         [SDKKinds.LuisRecognizer]: {
-          recognizer: { displayName: 'recognizer2' },
+          recognizer: { displayName: 'recognizer2', description: 'recognizer2' },
         },
       },
     };
 
     expect(mergePluginConfigs(config1, config2).uiSchema).toEqual({
       [SDKKinds.RegexRecognizer]: {
-        recognizer: { displayName: 'recognizer1' },
+        recognizer: { displayName: 'recognizer1', description: 'recognizer1' },
       },
       [SDKKinds.LuisRecognizer]: {
-        recognizer: { displayName: 'recognizer2' },
+        recognizer: { displayName: 'recognizer2', description: 'recognizer2' },
       },
     });
   });

Composer/packages/extension-client/src/types/formSchema.ts

@@ -82,6 +82,8 @@ export type RecognizerSchema = {
   disabled?: boolean | ((shellData: ShellData, shellApi: ShellApi) => boolean);
   /** Display name used in the UI. Recommended to use function over static string to enable multi-locale feature. */
   displayName: UIOptionValue<string>;
+  /** Description used in the UI. describe the usage of this recognizer */
+  description: UIOptionValue<string>;
   /** An inline editor to edit an intent. If none provided, users will not be able to edit. */
   intentEditor?: FieldWidget | string;
   /** A function invoked with the form data to determine if this is the currently selected recognizer */

Composer/packages/integration-tests/cypress/integration/DesignPage.spec.ts

@@ -73,8 +73,10 @@ context('breadcrumb', () => {
   it('can create different kinds of triggers ', () => {
     cy.visitPage('Create');
     cy.findByTestId('DialogHeader-TestBot_TestSample').click();
-    cy.findByTestId('recognizerTypeDropdown').click();
-    cy.findByText('Regular expression recognizer').click();
+    // cy.findByText('Change').click();
+    cy.findByTestId('openRecognizerDialog').click();
+    cy.findByText('Regular expression').click();
+    cy.findByText('Done').click();
 
     //onintent trigger
     cy.findByTestId('DialogHeader-TestBot_TestSample').within(() => {