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

fix: do not use sdk schema for settings json editor #5964

Merged
merged 6 commits into from
Feb 26, 2021

Conversation

a-b-r-o-w-n
Copy link
Contributor

Description

Removes the schema used for settings editor which incorrectly was using the sdk schema.

Task Item

fixes #5905

hatpick
hatpick previously approved these changes Feb 25, 2021
srinaath
srinaath previously approved these changes Feb 26, 2021
@coveralls
Copy link

coveralls commented Feb 26, 2021

Coverage Status

Coverage increased (+0.01%) to 53.81% when pulling c1cb13e on abrown/fix-settings-schema into ae1ccdc on main.

@cwhitten cwhitten merged commit 4d39090 into main Feb 26, 2021
@cwhitten cwhitten deleted the abrown/fix-settings-schema branch February 26, 2021 19:40
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* do not use sdk schema for settings json editor

* remove unused variable

* remove unused import

Co-authored-by: Soroush <hatpick@gmail.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None of the properties in the Advanced Settings View are allowed
5 participants