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

feat: Merge bot-projects feature onto main #5066

Merged
merged 79 commits into from
Dec 3, 2020
Merged

Conversation

srinaath
Copy link
Contributor

@srinaath srinaath commented Dec 3, 2020

Merge bot project work into main

#minor

srinaath and others added 30 commits October 29, 2020 15:24
…oser into feature/bot-projects

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

# Conflicts:
#	Composer/packages/client/src/shell/useShell.ts
* Update projectTemplates.ts

* correct links and reading dialogs

* fix typecheck

* fixes from CR

* unify props shape across LG/LU/QnA pages

* fixes from CR
…oser into feature/bot-projects

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

# Conflicts:
#	extensions/localPublish/package.json
#	extensions/localPublish/yarn.lock
* feat: Notifications Page

* update the unit tests

* fix merge conflict

* update the name

* fix unit test

* Test skill (#2)

* Update en-US.json

* bring in stuff from the draft branch

* make deletion work

* add error/warning icons

* read notification map for state

* fix type-checking and start on unit tests

* add sampleDialog and fix more tests

* add showAll

* rename to onAllSelected because it's a callback

* update unit tests

* fix onSelect handling in ProjectTree

* Update qna.test.tsx

* Update design.test.tsx

* add unit test

* fixes from PR comments

* restore multibots to tree

* fix links from project tree

* fix wrong IDs in URL

* yarn.lock rebuild

* update yarn.lock files

* fix action card links

* Update DesignPage.tsx

* fix unit test

* add unit test to ExpandableNode

* update ProjectTree unit tests

* Update ExpandableNode.tsx

* add rootProjectId to triggerApi

* fix creation and deletion bugs

* change mocks folder name

* Load design page only after skill project is loaded

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* fixes from CR

* fix more tests

* fix: handle the case when selection is not empty (#4561)

Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>

* delete fd (#4556)

Co-authored-by: Soroush <sorgh@microsoft.com>

* fix more unit tests

* revert: bf-orchestrator lib integration (#4568)

This reverts commit 006a589.

Co-authored-by: Andy Brown <asbrown002@gmail.com>

* fix test typol

* ci: remove code scanning workflow (#4569)

* fix: Can not add properties in "SetProperties" action (#4572)

* bug fix

* pr comment

* pr comments

Co-authored-by: Ben Yackley <61990921+beyackle@users.noreply.github.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Co-authored-by: Srinaath Ravichandran <srravich@microsoft.com>
Co-authored-by: zeye <zeye@microsoft.com>
Co-authored-by: Soroush <hatpick@gmail.com>
Co-authored-by: Soroush <sorgh@microsoft.com>
Co-authored-by: taicchoumsft <61705609+taicchoumsft@users.noreply.github.com>
Co-authored-by: Andy Brown <asbrown002@gmail.com>
Co-authored-by: LouisEugeneMSFT <66701106+LouisEugeneMSFT@users.noreply.github.com>

* update url

* Revert "Test skill (#2)"

This reverts commit c401768.

* fix conflict

* fix e2e test

* fix unit test

* update the diagnostic page

* update the warning iconm

* fix lint

* add some unit tests

* fix conflict and update the navigation

* fix unit test

Co-authored-by: Ben Yackley <61990921+beyackle@users.noreply.github.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Co-authored-by: Srinaath Ravichandran <srravich@microsoft.com>
Co-authored-by: zeye <zeye@microsoft.com>
Co-authored-by: Soroush <hatpick@gmail.com>
Co-authored-by: Soroush <sorgh@microsoft.com>
Co-authored-by: taicchoumsft <61705609+taicchoumsft@users.noreply.github.com>
Co-authored-by: Andy Brown <asbrown002@gmail.com>
Co-authored-by: LouisEugeneMSFT <66701106+LouisEugeneMSFT@users.noreply.github.com>
Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>
Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>
…oser into feature/bot-projects

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

# Conflicts:
#	Composer/packages/client/src/components/TestController/TestController.tsx
#	Composer/packages/client/src/pages/setting/app-settings/AppSettings.tsx
#	Composer/packages/client/src/utils/hooks.ts
#	Composer/packages/client/src/utils/pageLinks.ts
* Update projectTemplates.ts

* Update navigation.test.tsx

* Update ProjectTree.tsx
* Update projectTemplates.ts

* Update navigation.test.tsx

* click last instance of bot/dialog name

* Update CreateNewBot.spec.ts

* update more tests
* Added schema for bot project support

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Added support to handle botproject space

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* More updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* temp

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Loading multiple projects working effeiciently

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Backup

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

creat new bot

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Bot project file

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* More updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* More updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* update the recoil to 0.0.13

* fix lint

* remove @type/recoil

* router work

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Bot Project file

creation, adding remote/local skills. Populate state management with rootbot and skills

* Migrate to old desing page and project tree

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Revert usage of React 16.13.1 to 16.13.0

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Revert "Revert usage of React 16.13.1 to 16.13.0"

This reverts commit 9aa46ca.

* Rename api handlers

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Add existing skill tests passing

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Add 1.0 botproject schema

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Fix typo

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Manifest tests updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

More updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Revert publish console logs

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

All unit tests added

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Removed code for bot migration

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Server side tests added

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated schema url to the correct one

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Fix lint and update schema url on echobot

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* botstates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

More file util tests added

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Support for exporting .botproj file

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Clean up paths

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Spellings cleanup

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated gitignore

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Typo fixed

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Set project ignore validation to true while opening remote skill

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Following sdk.schema approach to specificy object type using ref

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Runtime start first pass

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

skills update

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Migration of old bot wrapped up

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Runtime start updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Bot load status

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Bot status indicator

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Notification selector

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update to settings

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Working bot status indicator

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update to start component

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Stable LocalBotRuntime operations

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* add/remote skill sync to *.botproj file

* relative path support bp file

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Path set

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Fix schema

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Removed workspace references.

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Relative path support

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* creation flow of add/open skill bot, remote skill

* Update operations on Bot project file

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated runtime code

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* clean up

* Start bots panel first pass completed

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated tests for removing workspace reference

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Removed manifest update operation

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Cleared logs

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Remove skill references

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Remove skill references

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Tested

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Remove unused references

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update references

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated runtime experience

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* repair broken link modal

* update the recognizer types generate function

* repair broken skill link

* Safe removal of skills

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Resolve bugs with $kind

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* check LUIS & QnA key

* add diagnostics to botProjectSpaceSelector

* diagnostics in projectree

* safely remove unused refs

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Handle select skill dialog tests

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

All tests passing

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Removed logs

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Skill utils updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* diagnostics style in project tree

* context menu in project tree

* update

* Added more unit tests

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Added back file to get it back to main version

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Bot status tracker operation

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

More test updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* More tests added

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Moved test files

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Binding skill host endpoint on bot start

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Added aria labels

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Header tests updated

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* cleaned up tests and rebased main

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* yarn update and header test

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* PR cleanup before moving from draft status

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* package all setting check into botindexer.validate

* update

* Migration completed

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated yarn.lock on vacore

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Revert azure publish yarn lock

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* remove fit from test

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update settings on removing a skill

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Logic update for skill deletion

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Add get port to server

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Start root bot if skills are not present

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* trigger dispatcher

* context menu button enable

* start/stop bot in project tree

* edit/create manifest

* Create/edit skill manifest modal

* clean up  qnaKbUrls

* creation flow in design page

* update broken skill repair modal

* feat: local bot controller

* fet: Add test in emulator column

* Updated select skill dialog to disable controls if no skill

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* open emulator test

* Added fnd by role

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* open Emulator test updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update to skills Utils

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* update error message style

* Support skillhost endpoint for remote skills

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* edit skill manifest in edit mode

* fix menu flash

* Port set

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Update yarn lock

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* remote skill manifest editor

* refine broken bot

* fixed styles

* tests

* tests

* Unit tests completed

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* resolve merge conflict

* navigation and visual editor for remote skill

* lint fix

* resolve merge conflict

* update

* update

* get bot diagnostics when load bot

* update

* move skillsProjectIdSelector to selectors/skills

* navigate to skill bot, update test

* feat: Property Editor Header when botname is selected in Project tree (#4771)

* Support undo/redo for skill

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Poprety Editor commits

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Adaptive form updates

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* PropertyEditorHeader test

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Updated tests

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* PropertyEditor wrapped up

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Removed unused props

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Co-authored-by: Srinaath Ravichandran <srravich@microsoft.com>

* clean up

* move PropertyEditorHeader.test.tsx to client/design

* clean up

* clean up qnaKbUrls

* resolve merge conflict

* clean up

* clean

* validation from settings

* ut for trigger dispatcher

* fix e2e tests and update the dialog create logic

* fix unit tests

* fix create bot error

* change the warning and error position

* fix navigate error when create a new dialog

* update

* update link

* Header update

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* Adjust styles

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* bot check filter empty qna/lu files

* context menu in project tree header

* update

* Updated styles

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

Co-authored-by: Srinaath Ravichandran <srravich@microsoft.com>
Co-authored-by: Leilei Zhang <leilzh@microsoft.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Co-authored-by: Srinaath Ravichandran <srinaath27@gmail.com>
Co-authored-by: TJ <tj.durnford@microsoft.com>
Co-authored-by: Ben Yackley <61990921+beyackle@users.noreply.github.com>
* setting page UX

* update UX

* refine css

* update url

* update css and fix localStorage bug

* refactor and refine css

* add custom error style and fix bugs of language settings

* localstorage migration

* fix delete button hidden bug

* update css

* update Composer Setting page

* save tmp code about the callout switching  active language in design page

* changing locale button on Header

* lint fix

* lint fix

* handle comments

* save tmp code

* handle comments

* save tmp code

* add test case

* add test cases

* add test cases

* fix bugs and update router component

* handle comments

* handle comments

* handle comments

* update qna end point key after update qna subscription key

* code style

* set defaultLanguage as active language if active language is deleted

* fix UT

* remove useless files

* save tmp code

* sync on root key

* fix bug

* code refine

* fix UT

* fix bug

* Refactoring function calls

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

* lint fix

Co-authored-by: Srinaath Ravichandran <srinaath27@gmail.com>
Co-authored-by: Srinaath Ravichandran <srravich@microsoft.com>
* Update projectTemplates.ts

* Update navigation.ts

* Update navigation.test.tsx
…oser into feature/bot-projects

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

# Conflicts:
#	Composer/packages/lib/shared/__tests__/skillUtils/index.test.ts
Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>
* Update projectTemplates.ts

* Update navigation.test.tsx

* add logic for new highlighting with menu open

* fix color logic
* chore: update the navigation link for setting page

* clean the code

* update the bot diagnostics

* fix diagnostic page link error

Co-authored-by: Soroush <hatpick@gmail.com>
…oser into feature/bot-projects

Signed-off-by: Srinaath Ravichandran <srravich@microsoft.com>

# Conflicts:
#	Composer/packages/client/src/recoilModel/atoms/appState.ts
#	Composer/packages/client/src/recoilModel/dispatchers/project.ts
#	Composer/packages/client/src/recoilModel/dispatchers/utils/project.ts
#	Composer/packages/client/src/router.tsx
* Update projectTemplates.ts

* Update navigation.test.tsx

* Update treeItem.tsx
Srinaath Ravichandran and others added 5 commits December 2, 2020 13:13
@srinaath srinaath changed the title TEST- DO NOT MERGE Test to make sure there is a linear history on the feature branch- DO NOT MERGE Dec 3, 2020
@srinaath srinaath changed the title Test to make sure there is a linear history on the feature branch- DO NOT MERGE chore: Test to make sure there is a linear history on the feature branch- DO NOT MERGE Dec 3, 2020
@srinaath srinaath changed the title chore: Test to make sure there is a linear history on the feature branch- DO NOT MERGE chore: Merge bot-projects feature onto main Dec 3, 2020
@coveralls
Copy link

coveralls commented Dec 3, 2020

Coverage Status

Coverage increased (+0.1%) to 55.361% when pulling 412e29f on feature/bot-projects-new into 2fb9f59 on main.

@srinaath srinaath added this to the R11 milestone Dec 3, 2020
@srinaath srinaath added BotProjects Group all Bot projects tickets and removed status: on hold labels Dec 3, 2020
@srinaath srinaath merged commit 05aa579 into main Dec 3, 2020
@srinaath srinaath deleted the feature/bot-projects-new branch December 3, 2020 02:34
@srinaath srinaath changed the title chore: Merge bot-projects feature onto main feat: Merge bot-projects feature onto main Dec 15, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BotProjects Group all Bot projects tickets
Projects
None yet
Development

Successfully merging this pull request may close these issues.